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

[breaking changes] update syntax and CLI #546

Merged
merged 2 commits into from
Dec 19, 2023
Merged

[breaking changes] update syntax and CLI #546

merged 2 commits into from
Dec 19, 2023

Conversation

AltGr
Copy link
Contributor

@AltGr AltGr commented Dec 7, 2023 •
edited
Loading

@AltGr AltGr force-pushed the breaking branch 2 times, most recently from 5b08023 to 6092cf3 Compare December 8, 2023 14:39
denismerigoux
denismerigoux approved these changes Dec 11, 2023
View reviewed changes
Copy link
Contributor

@denismerigoux denismerigoux left a comment

Choose a reason for hiding this comment

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

Thanks @AltGr ! I'm okay with these breaking changes but we should :

  • publish a new version 0.9.0 of the opam package with a changelog containing this
  • make sure clerk binary is included in the catala opam package (it's not the case at the moment)

examples/droit_successions/tests/tests_droit_succession.catala_fr Outdated
@@ -69,20 +69,112 @@ champ d'application Test4:

```catala-test-inline
$ catala Interpret -s Test1
[RESULT] Computation successful!
[ERROR]
Copy link
Contributor

Choose a reason for hiding this comment

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

This test should be updated.

@denismerigoux
Copy link
Contributor

Let's merge this quickly ?

@AltGr
Copy link
Contributor Author

AltGr commented Dec 19, 2023

As soon as the tests pass :)

@AltGr AltGr merged commit 50cb0db into CatalaLang:master Dec 19, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@denismerigoux denismerigoux denismerigoux approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Renaming "collection" as "list"
2 participants
@AltGr @denismerigoux