Skip to content

Fix/5 2 and minors#2

Merged
abdratsd merged 3 commits into
ainetus:mainfrom
ahmerique:fix/5-2-and-minors
Jul 29, 2025
Merged

Fix/5 2 and minors#2
abdratsd merged 3 commits into
ainetus:mainfrom
ahmerique:fix/5-2-and-minors

Conversation

@ahmerique
Copy link
Copy Markdown

Fix following issues using the recommended fixes:
IRT-SystemX#5
IRT-SystemX#2

Minor changes to the README
Force files read by docker to use end of line character "lf" to be compatible with wsl

Update readme
Force eol character to LF for files transfered to a docker volume to prevent compatibility issues in windows docker
@ahmerique
Copy link
Copy Markdown
Author

@mjothy
@abdratsd

@mjothy
Copy link
Copy Markdown

mjothy commented Jul 8, 2025

Looks good, good to add a gitattribute for enforcing eol issues.

@mjothy
Copy link
Copy Markdown

mjothy commented Jul 8, 2025

@brunolemetayer : could you confirm that the new line names in the format {line_or_to_subid}:{line_ex_to_subid}:{name_line} is ok ? (for issue IRT-SystemX#2 )

@brunolemetayer
Copy link
Copy Markdown

@mjothy yes !

@ahmerique ahmerique force-pushed the fix/5-2-and-minors branch from bc23982 to f846f8e Compare July 28, 2025 12:47
@abdratsd abdratsd merged commit 9ca70f5 into ainetus:main Jul 29, 2025
abdratsd pushed a commit that referenced this pull request Oct 22, 2025
…ine name as {origin_subid}:{origin_extremity}:{line_name}
abdratsd pushed a commit that referenced this pull request Oct 22, 2025
…ine name as {origin_subid}:{origin_extremity}:{line_name}
abdratsd pushed a commit that referenced this pull request Oct 22, 2025
…ine name as {origin_subid}:{origin_extremity}:{line_name}
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.

4 participants