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

ci: add test matrix #1234

Merged
merged 11 commits into from
Oct 1, 2024
Merged

ci: add test matrix #1234

merged 11 commits into from
Oct 1, 2024

Conversation

bonjourmauko
Copy link
Member

Supersedes #1168

@bonjourmauko bonjourmauko added the kind:discovery Issue requires discovery: value, ux and tech label Sep 27, 2024
@bonjourmauko bonjourmauko added this to the Improved testing milestone Sep 27, 2024
@bonjourmauko bonjourmauko self-assigned this Sep 27, 2024
@bonjourmauko bonjourmauko mentioned this pull request Sep 27, 2024
Closed
@coveralls
Copy link

Coverage Status

coverage: 81.462% (+7.0%) from 74.491%
when pulling e96c619 on ci/matrix
into 7d451a1 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

coverage: 81.462% (+7.0%) from 74.491%
when pulling e96c619 on ci/matrix
into 7d451a1 on master.

@coveralls
Copy link

coveralls commented Sep 27, 2024 •
edited
Loading

Coverage Status

coverage: 81.654% (+7.1%) from 74.577%
when pulling 2f7da3f on ci/matrix
into 1e894f4 on master.

@bonjourmauko bonjourmauko marked this pull request as ready for review September 27, 2024 16:36
@bonjourmauko bonjourmauko requested review from benoit-cty and a team September 27, 2024 16:36
@bonjourmauko bonjourmauko added kind:ci Continuous ops, integration & deployment and removed kind:discovery Issue requires discovery: value, ux and tech labels Sep 27, 2024
benoit-cty
benoit-cty reviewed Sep 30, 2024
View reviewed changes
Copy link
Contributor

@benoit-cty benoit-cty left a comment

Choose a reason for hiding this comment

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

Thanks, it's nice to have shorter files for workflows.

I have some questions.

.github/workflows/push.yaml Outdated Show resolved Hide resolved
.github/workflows/push.yaml Outdated Show resolved Hide resolved
.github/workflows/push.yaml Outdated Show resolved Hide resolved
.github/publish-git-tag.sh Outdated Show resolved Hide resolved
@bonjourmauko bonjourmauko force-pushed the ci/matrix branch 2 times, most recently from fb01b09 to 32dfe8c Compare September 30, 2024 17:15
@bonjourmauko bonjourmauko force-pushed the ci/matrix branch 20 times, most recently from 4baace2 to afca357 Compare October 1, 2024 17:48
@bonjourmauko bonjourmauko merged commit 0c3544a into master Oct 1, 2024
28 checks passed
@bonjourmauko bonjourmauko deleted the ci/matrix branch October 1, 2024 18:08
bonjourmauko added a commit that referenced this pull request Oct 2, 2024
@bonjourmauko
revert: "ci: add test matrix (#1234)"
17baccb 
This reverts commit 0c3544a, reversing
changes made to 1e894f4.
@bonjourmauko bonjourmauko mentioned this pull request Oct 4, 2024
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benoit-cty benoit-cty benoit-cty approved these changes

Assignees

@bonjourmauko bonjourmauko

Labels
kind:ci Continuous ops, integration & deployment
Projects
OpenFisca International
Status: Shipped
Milestone
Improved build/release workflow
Development

Successfully merging this pull request may close these issues.
3 participants
@bonjourmauko @coveralls @benoit-cty