Skip to content

Delete macOS comment from bootstrap script#120

Merged
HAEKADI merged 4 commits into
masterfrom
bootstrap
Oct 21, 2021
Merged

Delete macOS comment from bootstrap script#120
HAEKADI merged 4 commits into
masterfrom
bootstrap

Conversation

@HAEKADI
Copy link
Copy Markdown
Contributor

@HAEKADI HAEKADI commented Oct 21, 2021

Fix that should have been included in #115 to make sure the bootstrap script is executed correctly on macOS.
Find details here.

@HAEKADI HAEKADI changed the title Fix illegal byte sequence bug in bootstrap Fix illegal byte sequence bug in bootstrap script Oct 21, 2021
@HAEKADI HAEKADI requested a review from MattiSG October 21, 2021 10:28
@HAEKADI
Copy link
Copy Markdown
Contributor Author

HAEKADI commented Oct 21, 2021

The current bootstrap script works properly in my environment.

@HAEKADI HAEKADI self-assigned this Oct 21, 2021
Copy link
Copy Markdown
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.

Cannot reproduce locally, I trust @HAEKADI with having checked that solves the issue she reported 😉

@HAEKADI
Copy link
Copy Markdown
Contributor Author

HAEKADI commented Oct 21, 2021

@MattiSG After choosing another country name to check and changing directories. I am no longer able able to reproduce the error 🤔 I will use this PR to delete the comment altogether instead :) Sorry!

@HAEKADI HAEKADI changed the title Fix illegal byte sequence bug in bootstrap script Delete macOS comment from bootstrap script Oct 21, 2021
@HAEKADI HAEKADI requested a review from MattiSG October 21, 2021 10:42
Comment thread CHANGELOG.md
Comment on lines +3 to +8
### 3.13.1 - [#120](https://github.com/openfisca/country-template/pull/120)

* Technical improvement.
* Details:
- Fix illegal byte sequence bug in bootstrap script on macOS

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is outdated.

@HAEKADI HAEKADI requested a review from MattiSG October 21, 2021 10:47
@HAEKADI HAEKADI merged commit e12e1b6 into master Oct 21, 2021
@HAEKADI HAEKADI deleted the bootstrap branch October 21, 2021 10:48
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