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

[2/4] [test-built] Improve building workflow #1058

Closed
wants to merge 9 commits into from
Closed

[2/4] [test-built] Improve building workflow #1058

wants to merge 9 commits into from

Conversation

bonjourmauko
Copy link
Member

@bonjourmauko bonjourmauko commented Oct 1, 2021 •
edited

Fixes #1121
Depends on #1048
Depends on #1015
Depended upon by #1059

Non-technical changes

  • Moves building workflow to Makefile.

@bonjourmauko bonjourmauko added the kind:devops Continuous ops, integration & deployment label Oct 1, 2021
@bonjourmauko bonjourmauko added this to the Improved releases milestone Oct 1, 2021
@bonjourmauko bonjourmauko added this to Inbox in OpenFisca International via automation Oct 1, 2021
@bonjourmauko bonjourmauko added kind:roadmap A group of issues, constituting a delivery roadmap kind:theme A group of issues, directly tied to an OKR labels Oct 1, 2021
This was referenced Oct 1, 2021
Merged
Draft
@bonjourmauko bonjourmauko changed the base branch from master to fix-yaml-test-bug October 1, 2021 15:33
@bonjourmauko bonjourmauko requested review from MattiSG and a team October 1, 2021 15:34
@bonjourmauko bonjourmauko changed the title [2/3] Improve building workflow [2/3] [test-built] Improve building workflow Oct 1, 2021
@bonjourmauko bonjourmauko force-pushed the fix-build branch 2 times, most recently from ce67c6d to 0c4da49 Compare October 1, 2021 16:45
@bonjourmauko bonjourmauko changed the title [2/3] [test-built] Improve building workflow [2/4] [test-built] Improve building workflow Oct 1, 2021
@bonjourmauko bonjourmauko mentioned this pull request Oct 1, 2021
Draft
MattiSG
MattiSG reviewed Oct 5, 2021
View reviewed changes
Copy link
Member

@MattiSG MattiSG left a comment

Choose a reason for hiding this comment

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

There are too many competing PRs (#1050, #1057) trying to implement parts of #1040. We will go through them over a video call tomorrow with @maukoquiroga to have a common understanding of the roadmap 🙂

.circleci/config.yml Outdated Show resolved Hide resolved
Base automatically changed from fix-yaml-test-bug to master October 6, 2021 08:30
@MattiSG
Rename ci task to install build dependencies
7600bc4 
Co-authored-by: Matti Schneider <matti.schneider@beta.gouv.fr>
@bonjourmauko bonjourmauko removed kind:theme A group of issues, directly tied to an OKR kind:roadmap A group of issues, constituting a delivery roadmap labels Oct 7, 2021
@bonjourmauko bonjourmauko mentioned this pull request Dec 1, 2022
Open
@bonjourmauko bonjourmauko mentioned this pull request Dec 2, 2022
Open
@bonjourmauko
Copy link
Member Author

Superseded by #1168

OpenFisca International automation moved this from Inbox to Abandoned Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MattiSG MattiSG Awaiting requested review from MattiSG

Assignees
No one assigned
Labels
kind:devops Continuous ops, integration & deployment
Projects
OpenFisca International
  
Abandoned
Milestone
Improved releases
Development

Successfully merging this pull request may close these issues.

Release contains [dev] extra requires
2 participants
@bonjourmauko @MattiSG