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 deps #244

Merged
merged 2 commits into from Jan 19, 2022
Merged

update deps #244

merged 2 commits into from Jan 19, 2022

Conversation

jdroenner
Copy link
Member

  • I agree to follow the project's code of conduct.
  • I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

@lnicola
Copy link
Member

lnicola commented Jan 19, 2022

bors r+

bors bot added a commit that referenced this pull request Jan 19, 2022
244: update deps r=lnicola a=jdroenner

- [x] I agree to follow the project's [code of conduct](https://github.com/georust/gdal/blob/master/CODE_OF_CONDUCT.md).
- [ ] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.
---



Co-authored-by: Johannes Drönner <droenner@mathematik.uni-marburg.de>
@lnicola
Copy link
Member

lnicola commented Jan 19, 2022

It's broken, isn't it?

@jdroenner
Copy link
Member Author

bors? I guess the new CI is too much for bors.

@lnicola
Copy link
Member

lnicola commented Jan 19, 2022

I guess https://github.com/georust/gdal/blob/master/bors.toml needs to be updated with the job names from the workflow:

image

@jdroenner
Copy link
Member Author

uh good catch

@bors
Copy link
Contributor

bors bot commented Jan 19, 2022

Canceled.

@jdroenner
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Jan 19, 2022
@lnicola
Copy link
Member

lnicola commented Jan 19, 2022

That won't work, AFAIK you need to update bors.toml on master before using it.

@bors
Copy link
Contributor

bors bot commented Jan 19, 2022

try

Build succeeded:

@jdroenner
Copy link
Member Author

jdroenner commented Jan 19, 2022

thats strange. The CI logic is not always clear to me

@lnicola
Copy link
Member

lnicola commented Jan 19, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Jan 19, 2022

Build succeeded:

@bors bors bot merged commit 414f924 into master Jan 19, 2022
@ChristianBeilschmidt ChristianBeilschmidt deleted the update-dependencies branch September 2, 2022 15:29
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

2 participants