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

Interpolation of files flag #222

Open
jaydorsey opened this issue Jan 14, 2025 · 1 comment
Open

Interpolation of files flag #222

jaydorsey opened this issue Jan 14, 2025 · 1 comment
Assignees
Labels
Medium Medium Priority Issues (to be fixed or re-evaluated in 3 months

Comments

@jaydorsey
Copy link

I have a config file that has a setup like this:

      - codecov/upload:
          files: coverage-results/backend-${CIRCLE_NODE_INDEX}.xml
          disable_search: true

It appears that ${CIRCLE_NODE_INDEX} isn't getting interpolated. I expected it to, so I'm not sure if that's a bug, a miss on my part (not seeing the right documentation), or an undocumented behavior.

The documentation that is there says files is "Comma-separated list of explicit files to upload. These will be added to the coverage files found for upload. If you wish to only upload the specified files, please consider using disable-search to disable uploading other files."

When I upgraded our orb recently and we switched to using files, I also set disable_search because we have a pretty explicit file name we wanted to include (it just relies on that ENV, which is set)

The actual behavior seems to be that the files is not interpolated/expanded (but some glob behaviors might work per other comments)

I don't know enough about the orb behavior themselves to know if this can be improved to allow for interpolation, but if not, would you be open to a pull request to improve the documentation on the files behavior so it's clear whether interpolation or globbing are supported by that string?

Note: I worked around this by removing disable_search

@thomasrockhu-codecov thomasrockhu-codecov self-assigned this Jan 17, 2025
@thomasrockhu-codecov thomasrockhu-codecov added the Medium Medium Priority Issues (to be fixed or re-evaluated in 3 months label Jan 17, 2025
@thomasrockhu-codecov
Copy link
Contributor

@jaydorsey definitely open to a PR. AFAIK, the files argument is not going to interpolate ${CIRCLE_NODE_INDEX}. I can dig into what we might have to do to support that behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Medium Medium Priority Issues (to be fixed or re-evaluated in 3 months
Projects
None yet
Development

No branches or pull requests

2 participants