Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Testing #744

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Testing #744

wants to merge 1 commit into from

Conversation

gregmagolan
Copy link
Member


Changes are visible to end-users: yes/no

  • Searched for relevant documentation and updated as needed: yes/no
  • Breaking change (forces users to change their own code or config): yes/no
  • Suggested release notes appear below: yes/no

Test plan

  • Covered by existing test cases
  • New test cases added
  • Manual testing; please provide instructions so we can reproduce:

Copy link

aspect-workflows bot commented Sep 13, 2024

Test

All tests were cache hits

225 tests (100.0%) were fully cached saving 7m 6s.


Lint

0 issues require fixes
2 issues can be fixed automatically

💡 Auto-fix linting errors with:

bazel lint //docs:docs_delivery //docs:docs_delivery_only_on_change --fix

Buildifier      Format

@gregmagolan gregmagolan force-pushed the workflows_smoke branch 5 times, most recently from 234d1c9 to 2691686 Compare September 16, 2024 01:12
set -o errexit -o nounset -o pipefail

msg="Demostration delivery target"
echo $msg

Choose a reason for hiding this comment

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

ShellCheck provided a suggestion

Suggested change
echo $msg
echo "${msg}"

set -o errexit -o nounset -o pipefail

msg="Demostration delivery target"
echo $msg

Choose a reason for hiding this comment

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

ShellCheck provided a suggestion

Suggested change
echo $msg
echo "${msg}"

@gregmagolan gregmagolan force-pushed the main branch 2 times, most recently from 84270db to a7ed65a Compare October 4, 2024 01:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant