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

Adds CoC badge per https://github.com/edgi-govdata-archiving/overview… #44

Merged
merged 1 commit into from Mar 5, 2019

Conversation

Frijol
Copy link
Contributor

@Frijol Frijol commented Feb 1, 2019

@lightandluck
Copy link
Member

Hmmm, this is breaking CircleCI build and I don't understand why

@Frijol
Copy link
Contributor Author

Frijol commented Feb 5, 2019

Looks like it failed on Zoom API credentials (have they been updated/this build would fail on any PR?)
@Mr0grog do you know?

@Mr0grog
Copy link
Member

Mr0grog commented Feb 12, 2019

Oops, sorry I didn’t see this last week. @Frijol you made your PR from a fork instead of a branch on this repo, so as a security measure, Circle won’t give you access to secret env vars.

@Frijol
Copy link
Contributor Author

Frijol commented Feb 12, 2019

@Mr0grog does that mean this is safe to merge despite the failed check?

@Mr0grog
Copy link
Member

Mr0grog commented Feb 13, 2019

Sure! More generally, it means you’re always going to have this problem, which sometimes won’t be such a non-issue, when you make a PR from a fork on this repo. (So in the future, please don’t PR to this repo from a fork.)

@Frijol Frijol merged commit 256e43e into edgi-govdata-archiving:master Mar 5, 2019
@Frijol Frijol deleted the patch-1 branch March 5, 2019 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants