Skip to content

Export TEST_CASE_RUN#5499

Merged
ferrarimarco merged 1 commit intomainfrom
export-test-case-run
Apr 8, 2024
Merged

Export TEST_CASE_RUN#5499
ferrarimarco merged 1 commit intomainfrom
export-test-case-run

Conversation

@ferrarimarco
Copy link
Copy Markdown
Collaborator

Proposed changes

Export the TEST_CASE_RUN variable because subprocesses reference it when building the file list.

Fix #5483

Readiness checklist

In order to have this pull request merged, complete the following tasks.

Pull request author tasks

  • I included all the needed documentation for this change.
  • I provided the necessary tests.
  • I squashed all the commits into a single commit.
  • I followed the Conventional Commit v1.0.0 spec.
  • I wrote the necessary upgrade instructions in the upgrade guide.
  • If this pull request is about and existing issue,
    I added the Fix #ISSUE_NUMBER label to the description of the pull request.

Super-linter maintainer tasks

  • Label as breaking if this change breaks compatibility with the previous released version.
  • Label as either: automation, bug, documentation, enhancement, infrastructure.

@ferrarimarco ferrarimarco added bug Something isn't working O: backlog 🤖 Backlog, stale ignores this label labels Apr 8, 2024
@ferrarimarco ferrarimarco self-assigned this Apr 8, 2024
@ferrarimarco ferrarimarco changed the title fix: export test_case_run Export TEST_CASE_RUN Apr 8, 2024
- Export the TEST_CASE_RUN variable because subprocesses reference it
  when building the file list.
- Remove a duplicate file that we didn't catch because Jscpd was not
  running due to the TEST_CASE_RUN variable not being exported.

Fix #5483
@ferrarimarco ferrarimarco force-pushed the export-test-case-run branch from d6c55bf to 716d5df Compare April 8, 2024 17:10
@ferrarimarco ferrarimarco merged commit cb109f4 into main Apr 8, 2024
@ferrarimarco ferrarimarco deleted the export-test-case-run branch April 8, 2024 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working O: backlog 🤖 Backlog, stale ignores this label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JSCPD Doesn't Run Unless TEST_CASE_RUN is set

1 participant