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

Update README.md with CI issues notice #519

Merged
merged 1 commit into from
Mar 19, 2021
Merged

Update README.md with CI issues notice #519

merged 1 commit into from
Mar 19, 2021

Conversation

Temikus
Copy link
Member

@Temikus Temikus commented Mar 8, 2021

It really sucks that we have to do this but I see no other way rather than ask for help. I also realise that this org is not owned by us so before I put the notice up I want @plribeiro3000 and @geemus to take a look and see if you folks are ok with such a public plea.

Thank you.

CC @icco

@Temikus Temikus requested review from geemus, icco and plribeiro3000 March 8, 2021 08:25
@geemus
Copy link
Member

geemus commented Mar 8, 2021

I think it's probably fine given the context.

That being said, I know I've loosened my own stance on real tests over time (by necessity). There are quite a few providers that I don't have active accounts on any more. I usually try to mitigate this by using the mock tests, asking folks who do feature work or bug fixes to test it for real, and knowing that I can always rollback changes or releases if I really have to. It's not perfect, but similar to what you are running into here, sometimes there aren't obvious/easy others ways to address it.

@Temikus Temikus merged commit 0411601 into master Mar 19, 2021
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.

2 participants