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.

Sign up for GitHub

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

Jump to bottom

Refactor bundler test spec ci #124

Merged
merged 2 commits into from
May 13, 2023
Merged

Refactor bundler test spec ci #124

merged 2 commits into from
May 13, 2023

Conversation

zsluedem
Copy link
Collaborator

@zsluedem zsluedem commented May 13, 2023 •
edited
Loading

zsluedem#3 here is the test pr I made for myself

@zsluedem zsluedem requested a review from Vid201 May 13, 2023 05:39
@Vid201
Copy link
Member

Vid201 commented May 13, 2023

LGTM. One question: What is the main reason for RUST_LOG=aa_bundler=TRACE?

@zsluedem
Copy link
Collaborator Author

LGTM. One question: What is the main reason for RUST_LOG=aa_bundler=TRACE?

Just for debugging when tests failed.

@zsluedem zsluedem closed this May 13, 2023
@zsluedem zsluedem reopened this May 13, 2023
@Vid201 Vid201 merged commit e1cdb5a into silius-rs:main May 13, 2023
@zsluedem zsluedem deleted the ci-update branch May 15, 2023 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vid201 Vid201 Vid201 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zsluedem @Vid201