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

[Broker] Remove possible error case for system topic name checking #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

michaeljmarshall
Copy link
Owner

No description provided.

@michaeljmarshall michaeljmarshall force-pushed the test-ci-potential-errors branch 12 times, most recently from 00a3f84 to 8022f55 Compare April 20, 2021 06:01
@michaeljmarshall michaeljmarshall force-pushed the test-ci-potential-errors branch from 8022f55 to b0590ae Compare April 20, 2021 06:08
@@ -58,8 +58,11 @@ jobs:
- '*.md'
- '**/*.md'

- name: print env
run: echo ${{ steps.changes.outputs.all_count }} && echo ${{ steps.changes.outputs.docs_count }} && echo ${{ steps.changes.outputs.docs }} && echo ${{steps.changes.outputs.all_count > steps.changes.outputs.docs_count}}
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This echo shows some of the values that are interpolated by GitHub. View them in the action's logs.

- name: Cache local Maven repository
if: ${{ steps.changes.outputs.all_count > steps.changes.outputs.docs_count }}
if: ${{ steps.changes.outputs.all_count }} > ${{ steps.changes.outputs.docs_count }}
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this test runs, but the others don't.

@github-actions
Copy link

The pr had no activity for 30 days, mark with Stale label.

@github-actions github-actions bot added the Stale label Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant