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

feat(ci): switch to github action #210

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

BuonOmo
Copy link
Contributor

@BuonOmo BuonOmo commented Mar 12, 2024

@sferik json_pure tests are failing. I skept them as I don't understand what is the issue about. I think it is better to have tests like this to avoid further regressions and then look for the issue. I'll be less disponible next days, so here's a mergeable (IMHO) proposal

You'd have to activate Gihub Actions for the project, and branch protect master (and rename it ? <3)

@BuonOmo BuonOmo force-pushed the gh-action branch 2 times, most recently from d96af4e to 4de6dce Compare March 12, 2024 23:09
@sferik sferik merged commit 476383d into intridea:master Mar 13, 2024
@sferik
Copy link
Member

sferik commented Mar 13, 2024

Looks good! Thanks for the quick patch. I am able to create a main branch from master but I'm not able to change the default. Paging @intridea-shared to activate GitHub Actions and change the default branch!

@sferik
Copy link
Member

sferik commented Mar 13, 2024

Just merged this but noticed one more change that needs to be made: the build status badge in the README needs to be updated too.

Build Status

@BuonOmo
Copy link
Contributor Author

BuonOmo commented Mar 13, 2024

EDIT: PR done here #212

Arf I forgot it... Could you add it committing directly ?

main

[![CI](https://github.com/intridea/multi_json/workflows/Test/badge.svg)](https://github.com/intridea/_multijson/actions?query=workflow%3ATest+branch%3Amain+event%3Apush)

master

[![CI](https://github.com/intridea/multi_json/workflows/Test/badge.svg)](https://github.com/intridea/multi_json/actions?query=workflow%3ATest+branch%3Amaster+event%3Apush)

CI

I could do the PR once the branch is changed otherwise :)

@BuonOmo BuonOmo deleted the gh-action branch March 15, 2024 16:11
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