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

Spaces in Stage names #89

Open
tschwarzl opened this issue Jan 27, 2025 · 2 comments
Open

Spaces in Stage names #89

tschwarzl opened this issue Jan 27, 2025 · 2 comments
Labels
question Further information is requested

Comments

@tschwarzl
Copy link
Contributor

at the moment, it is possible to create stage names with spaces.

dso create stage '0103_test spaces'

We need to make sure, that this will not break anything downstream, because e.g.

dvc.yaml

will e.g. be compiled to:

stages:
  0103_test spaces:
    params:
      - dso.quarto
    deps:
      - src/0103_test spaces.qmd
    outs:
      - output
      - report/0103_test spaces.html
    cmd:
      - dso exec quarto .

Currently, a repro did work.

This ticket is a placeholder for discussion.

@tschwarzl tschwarzl added the question Further information is requested label Jan 27, 2025
@grst
Copy link
Collaborator

grst commented Jan 27, 2025

Thanks for brining that up! It's either way bad practice, so we should possibly enforce some contraints on the stage names

@tschwarzl
Copy link
Contributor Author

constraints might be the most practical solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants