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

chore: update spread test to latest release changes #147

Merged

Conversation

letFunny
Copy link
Collaborator

  • Have you signed the CLA?

@letFunny letFunny requested a review from cjdcordeiro July 12, 2024 10:42
tests/basic/task.yaml Outdated Show resolved Hide resolved
Simpler slice with mutation scripts that is consistent across versions
of Chisel and we not forsee changes to it in the near future.
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks :)

Copy link
Member

@rebornplusplus rebornplusplus left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@letFunny letFunny added Simple Nice for a quick look on a minute or two Priority Look at me first labels Jul 29, 2024
@rebornplusplus rebornplusplus mentioned this pull request Jul 30, 2024
1 task
@cjdcordeiro cjdcordeiro self-assigned this Aug 6, 2024
@cjdcordeiro cjdcordeiro merged commit 73e7a7c into canonical:main Aug 6, 2024
14 checks passed
@letFunny letFunny deleted the fix-spread-tests-ca-certificates branch October 17, 2024 08:37
zhijie-yang pushed a commit to zhijie-yang/chisel that referenced this pull request Nov 7, 2024
* also change slice to base-passwd_data

Simpler slice with mutation scripts that is consistent across versions
of Chisel and we do not foresee changes to it in the near future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority Look at me first Simple Nice for a quick look on a minute or two
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants