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.

Sign up for GitHub

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

Jump to bottom

[11.0][IMP] Move repository to github actions #392

Merged
merged 4 commits into from
Oct 12, 2023
Merged

[11.0][IMP] Move repository to github actions #392

merged 4 commits into from
Oct 12, 2023

Conversation

NL66278
Copy link
Contributor

@NL66278 NL66278 commented May 1, 2023

No description provided.

@NL66278 NL66278 force-pushed the 11.0-use-github-actions branch 2 times, most recently from 98166a4 to debf177 Compare May 1, 2023 14:03
@NL66278 NL66278 requested a review from astirpe May 3, 2023 12:05
@NL66278 NL66278 mentioned this pull request May 3, 2023
Closed
@NL66278
Copy link
Contributor Author

NL66278 commented May 3, 2023

@astirpe Need a combination of moving to pre-commit and github actions instead of travis, togheter with improvements to kvk and postcode modules to get everything merged and finally get all the other 11.0 PR's finalized.

This was referenced May 3, 2023
Closed
Merged
astirpe
astirpe approved these changes May 8, 2023
View reviewed changes
Copy link
Member

@astirpe astirpe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@NL66278
Copy link
Contributor Author

NL66278 commented Jul 12, 2023

/ócabot merge nobump

@NL66278
Copy link
Contributor Author

NL66278 commented Jul 12, 2023

@astirpe Apparently for some reason ocabot does not work for this PR. Shall I just merge it using the normal github action?

thomaspaulb
thomaspaulb approved these changes Jul 19, 2023
View reviewed changes
Copy link
Contributor

@thomaspaulb thomaspaulb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've removed the merge commits.

@thomaspaulb
Copy link
Contributor

Seems the l10n_nl_location_nuts item is now failing on this url not working, being defined here. Apparently this has gone elsewhere, and they're having the same problem in OCA/partner-contact#1398 as well.

Where did it go?

@NL66278 NL66278 merged commit 9c1679a into OCA:11.0 Oct 12, 2023
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astirpe astirpe astirpe approved these changes

@thomaspaulb thomaspaulb thomaspaulb approved these changes

Assignees
No one assigned
Labels
approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@NL66278 @thomaspaulb @astirpe @OCA-git-bot