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

[READY] - golden-tests: migrate to nix check #628

Merged
merged 2 commits into from
Oct 18, 2023

Conversation

sarcasticadmin
Copy link
Member

@sarcasticadmin sarcasticadmin commented Oct 7, 2023

Description of PR

Relates to: #627

Moving the openwrt golden test to nix checks.

Sidenote: Given how the work existing workflow via nix.check has grown maybe its best we rename the workflow.yml? I spun out an issue to look into this vs doing it in this PR: #630

Previous Behavior

  • Using golden tests via docker

New Behavior

  • Using golden tests via nix checks

Tests

  • gha tests passing

@sarcasticadmin sarcasticadmin force-pushed the rh/1696709124gomplate branch 2 times, most recently from 6ad4d8c to 62ed00b Compare October 13, 2023 01:33
@sarcasticadmin sarcasticadmin changed the title [WIP] - golden-tests: migrate to nix check [READY] - golden-tests: migrate to nix check Oct 14, 2023
Copy link
Collaborator

@owendelong owendelong 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, but that doesn't mean a lot... I trust you.

@owendelong owendelong merged commit e857c20 into master Oct 18, 2023
1 check passed
@owendelong owendelong deleted the rh/1696709124gomplate branch October 18, 2023 00:10
@sarcasticadmin
Copy link
Member Author

thanks for the merge @owendelong !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants