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

[PSyData] Driver is using ":" in filename #2444

Closed
hiker opened this issue Dec 18, 2023 · 0 comments · Fixed by #2445
Closed

[PSyData] Driver is using ":" in filename #2444

hiker opened this issue Dec 18, 2023 · 0 comments · Fixed by #2445

Comments

@hiker
Copy link
Collaborator

hiker commented Dec 18, 2023

":" in filenames are inconvenient, they stop tab-file-name completion (since the ":" must be escaped (at least on my WSL). It is also harder to use these names in Makefiles (since the ":" otherwise make the filenames appear to be a rule, so all":" need to be escaped).
hiker added a commit that referenced this issue Dec 19, 2023
@hiker
#2444 Use '-' instead of ':' to separate region and invoke counters.
287ab26
@hiker hiker mentioned this issue Dec 19, 2023
Merged
hiker added a commit that referenced this issue Feb 22, 2024
@hiker
#2444 Fixed usage of DeferredType.
4980605
hiker added a commit that referenced this issue Apr 9, 2024
@hiker
#2444 Fixed failing test.
e5692d4
hiker added a commit that referenced this issue Jul 2, 2024
@hiker
#2444 Fixed old references to ':', which is now '-'.
626b6c6
hiker added a commit that referenced this issue Jul 2, 2024
@hiker
#2444 Removed references to NEMO API, and highlighted lines in exampl…
0469a9a 
…e code.
hiker added a commit that referenced this issue Jul 2, 2024
@hiker
#2444 Fixed incorrect output in README.
16da788
hiker added a commit that referenced this issue Jul 2, 2024
@hiker
#2444 Improved documentation.
1a3a4bf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

2444 use dash instead of colon stfc/PSyclone
1 participant
@hiker