Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow single repositories to be excluded via config #172

Open
Confectrician opened this issue Nov 27, 2021 · 7 comments
Open

Allow single repositories to be excluded via config #172

Confectrician opened this issue Nov 27, 2021 · 7 comments

Comments

@Confectrician
Copy link

Confectrician commented Nov 27, 2021

Hi all,

We are using the dco app enabled for a whole organization, but wanted to exclude the DCO condition "only" for our documentation repository.
To do this currently, we would need to remove the global acess on the configuration and then add each needed repository on its own.
Also this would force us to maintain this for new repositories each time.

My suggestion would be to add configuration parallel to the members required one, which lets exclude single repositories by placing the dco configuration file in that repository.

I did a quick shot for this in my fork.
You can find it here: Confectrician@220b092

If you like the idea, i can provide a PR of course.

Best Regards
Jerome

@Confectrician Confectrician changed the title Allow single repositories to be excluded via vonfig Allow single repositories to be excluded via config Nov 27, 2021
@gr2m
Copy link
Contributor

gr2m commented Nov 27, 2021

Hi Jerome,

I feel like the best solution would be if GitHub implemented a feature to exclude repositories from a global installation. Could you send a support request at https://support.github.com/contact?

If that won't work, I think I'd prefer this configuration:

require: false

That you would put in the repository's .github/dco.yml file to overwrite the global config

@Confectrician
Copy link
Author

I have added a request via the contact formular.
Do you have any impression on how long it takes to get an answer for something like this?

@gr2m
Copy link
Contributor

gr2m commented Nov 27, 2021

usually within 1-2 workdays, you should get an answer by end of day Tuesday. By then we should know if GitHub is working on such a feature or not

@Confectrician
Copy link
Author

Still no response, but there isn't even a ticket available in my support view.
Do you think something went wrong maybe?

@gr2m
Copy link
Contributor

gr2m commented Dec 3, 2021

I filed a request with support, you should have gotten it in cc

@gr2m
Copy link
Contributor

gr2m commented Dec 6, 2021

just for reference for everyone interested, here is what we heard back from support

Thanks for reaching out and sharing your thoughts! I agree that having an exclusion list would be a neat solution.

Is that something you are working on or consider adding?

There is an open feature request for this with the Apps engineering team and I have added your report as a +1 on the internal issue. I don't have a timeline when it will be worked on but this is currently in good hands.

As it's not already in the works, I don't expect it to happen anytime soon, there was not a lot of activity on GitHub App related features lately

I suggest we leave the issue open to see if others are interested in it as well

@Confectrician
Copy link
Author

Fine for me.
We have done the "regular" config wy for the openHAB organization meanwhile and included every repo besides the documentation.

Not really satisfying, but it's working.
Wa can still add the config value as a workaround, if there is some more interest in this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants