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.

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

Allow mixing project and dlt file arguments directly #403

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

tohammer
Copy link
Contributor

Decide which one it is based on extension. Simplifies usage from command line when dlt and project files are grouped in directories.

Decide which one it is based on extension. Simplifies usage from
command line when dlt and project files are grouped in directories.

Signed-off-by: Tobias Hammer <[email protected]>
@tohammer tohammer changed the title Allow mixing project and dlt files directly Allow mixing project and dlt file arguments directly Dec 20, 2023
@alexmucde alexmucde self-requested a review January 19, 2024 08:44
@alexmucde alexmucde self-assigned this Jan 19, 2024
@alexmucde alexmucde added this to the Release v2.26.0 milestone Jan 19, 2024
@alexmucde
Copy link
Collaborator

@tohammer Please can you also here update the option documentation?
I think the options -l and -p are not needed anymore, DLT files (incl. multiple files) and project files can be now loaded without options.

@alexmucde alexmucde assigned tohammer and unassigned tohammer and alexmucde Jan 19, 2024
@tohammer
Copy link
Contributor Author

@alexmucde Should i remove the -l and -p options completely or keep them in code for compatibility? (and ofc will update release info/docsaccordingly)

@alexmucde
Copy link
Collaborator

@tohammer You can remove them completely

@alexmucde
Copy link
Collaborator

@tohammer I will merge now you commit and do the requested changes by my own, because i need further modifications.

@alexmucde alexmucde merged commit 1a762cf into COVESA:master Jan 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants