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/nightly-2020-02-27 #2314

Closed
wants to merge 7 commits into from

Conversation

martell
Copy link
Contributor

@martell martell commented Feb 28, 2020

With some of the other PRs coming in like #2248 #2257 I have been trying to maintain a fork with both of them in the same branch to see what the world would be like.

In order to simplify some things I thought it would be a good time to upstream the update to nightly-2020-02-27

This currently depends on #2269 ( dc3dc23 is the only real commit here )
I'm not sure what the process for picking a nightly is but I hope this is reasonable.

@martell martell force-pushed the update/nightly-2020-02-27 branch 2 times, most recently from 279c169 to dc3dc23 Compare February 28, 2020 07:16
@martell
Copy link
Contributor Author

martell commented Feb 28, 2020

hmm, what is interesting is that is passes when it contains #2248
I'll have to look at the specifics of why that is the case but I do believe that will be merged soon anyway?

@weiznich
Copy link
Member

weiznich commented Mar 2, 2020

#2248 will be merged as soon as we get at least another review on that PR, especially on the documentation.

weiznich and others added 4 commits March 23, 2020 22:36
If that actually works I'm proposing moving all of our ci/automation
to this service, because of:

* Integration with github seems to be vastly better
    + No need to click n times to go to logs
    + Rerun a failed job is easier
    + Maybe? Inline error messages in PR's
* Error messages are automatically extracted
* I think they have 20 free concurrent jobs, azure has only 10 (so
faster CI)

(Most of the config is copied from wundergraph)
@martell
Copy link
Contributor Author

martell commented Mar 24, 2020

super-seeded by #2339

@martell martell closed this Mar 24, 2020
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