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

build: add first time setup script #63

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

build: add first time setup script #63

wants to merge 6 commits into from

Conversation

bonjourmauko
Copy link
Member

@bonjourmauko bonjourmauko commented Oct 22, 2024 •
edited
Loading

Depends on #62
Depends on openfisca/openfisca-setup-builder#6

  • Technical improvement
  • Impacted areas: src/first_time_setup
  • Details:
    • Add first-time-setup.sh
    • Add unit-tested setup builder build.sh

First Time Setup

Screenshot 2024-10-22 at 18 02 45 Screenshot 2024-10-22 at 18 03 04

@bonjourmauko bonjourmauko added the kind:build Changes that affect the build system or external dependencies label Oct 22, 2024
@bonjourmauko bonjourmauko requested review from a team October 22, 2024 15:55
@bonjourmauko bonjourmauko self-assigned this Oct 22, 2024
@bonjourmauko bonjourmauko changed the base branch from master to build/adopt-src-layout October 22, 2024 15:55
@bonjourmauko bonjourmauko mentioned this pull request Oct 22, 2024
Merged
7 tasks
Base automatically changed from build/adopt-src-layout to master October 23, 2024 08:34
@bonjourmauko bonjourmauko force-pushed the build/add-first-time-setup branch 4 times, most recently from 8e9981b to 9dbb833 Compare October 23, 2024 15:55
@benoit-cty
Copy link
Contributor

Hello,
I got an error:

[λ] Initialise git repository...
./first-time-setup.sh: line 687: setup::init_repository: command not found
[i] Exiting, bye!

@bonjourmauko
Copy link
Member Author

@benoit-cty It is fixed now.

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

@verbman verbman Awaiting requested review from verbman

@HAEKADI HAEKADI Awaiting requested review from HAEKADI

At least 1 approving review is required to merge this pull request.

Assignees

@bonjourmauko bonjourmauko

Labels
kind:build Changes that affect the build system or external dependencies
Projects
OpenFisca International
Status: Inbox
Milestone
No milestone
Development

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