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

Switch from FLAP to fArgParse #1756

Closed
tclune opened this issue Oct 24, 2022 · 4 comments
Closed

Switch from FLAP to fArgParse #1756

tclune opened this issue Oct 24, 2022 · 4 comments
Assignees
@mathomp4
Labels
0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 📈 MAPL3 MAPL 3 Related ❗ High Priority This is a high priority PR 🎁 New Feature This is a new feature ⌛ Long Term Long term issues

Comments

@tclune
Copy link
Collaborator

tclune commented Oct 24, 2022

Long time coming. (@JulesKouatchou )
@tclune tclune added 🎁 New Feature This is a new feature 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. ❗ High Priority This is a high priority PR labels Oct 24, 2022
@mathomp4 mathomp4 mentioned this issue Oct 25, 2022
Merged
7 tasks
@mathomp4 mathomp4 added the 📈 MAPL3 MAPL 3 Related label Nov 14, 2022
@mathomp4 mathomp4 reopened this Feb 9, 2023
@mathomp4
Copy link
Member

mathomp4 commented Feb 9, 2023

Reopening this as we only added an fArgParse parser, but did not actually change anything.

I'll work on getting this maybe on MAPL3, though part of me would like to even change MAPL2 to use fArgParse and remove FLAP. But to do that might require some Bad Things™ that @tclune may not approve of. We'll see.

@mathomp4 mathomp4 mentioned this issue Feb 9, 2023
Merged
12 tasks
@stale
Copy link

stale bot commented Apr 10, 2023

This issue has been automatically marked as stale because it has not had recent activity. If there are no updates within 7 days, it will be closed. You can add the "long term" tag to prevent the Stale bot from closing this issue.

@stale stale bot added the ❄️ Stale This issue has been marked stale label Apr 10, 2023
@mathomp4 mathomp4 added the ⌛ Long Term Long term issues label Apr 10, 2023
@stale stale bot removed the ❄️ Stale This issue has been marked stale label Apr 10, 2023
@mathomp4
Copy link
Member

We are almost ready for this! We just need to move to Baselibs 7.11.0...

@mathomp4 mathomp4 mentioned this issue Apr 11, 2023
Merged
7 tasks
@mathomp4 mathomp4 mentioned this issue Jun 8, 2023
Merged
7 tasks
@tclune
Copy link
Collaborator Author

tclune commented Apr 24, 2024

MAPL3 won't use either one most likely. Closing.

@tclune tclune closed this as completed Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mathomp4 mathomp4

Labels
0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 📈 MAPL3 MAPL 3 Related ❗ High Priority This is a high priority PR 🎁 New Feature This is a new feature ⌛ Long Term Long term issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mathomp4 @tclune