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

Add a demo for stack restack #399

Merged
merged 1 commit into from
Aug 21, 2024
Merged

Add a demo for stack restack #399

merged 1 commit into from
Aug 21, 2024

Conversation

draftcode
Copy link
Contributor

@draftcode draftcode commented Aug 19, 2024 •
edited
Loading

No description provided.

@aviator-app Aviator app
Copy link
Contributor

aviator-app bot commented Aug 19, 2024 •
edited
Loading

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.

See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@aviator-app Aviator app
Copy link
Contributor

aviator-app bot commented Aug 19, 2024

FlexReview Summary

Based on the code complexity and the author's expertise score, these are the suggested reviewers:

  • @alex-aviator (current review load: 2)

See the list of alternate reviewers in the detailed breakdown below.

Detailed Breakdown Author’s expertise score for the modified files:
  • demos/stack_restack/stack_restack.mp4 (0.00)
  • demos/stack_restack/stack_restack.tape (0.00)
  • demos/test_utils.sh (1.00)
Files Reviewers
demos/test_utils.sh † @alex-aviator (score: 1.00, current review load: 2)

† Indicates that the file doesn't need an expert review. (?)

See full breakdown of the reviewers on the Aviator webapp.

@aviator-app Aviator app
Copy link
Contributor

aviator-app bot commented Aug 21, 2024

This pull request failed to merge: some CI status(es) failed. Once the issues are resolved, remove the blocked label and re-queue the pull request. Note that the pull request will be automatically re-queued if it has the mergequeue label.

Failed CI(s): golangci-lint

@aviator-app aviator-app bot added the blocked label Aug 21, 2024
draftcode added a commit that referenced this pull request Aug 21, 2024
@draftcode
Use the golangci-lint GitHub Action
32af0a1 
It seems that there's a change in golangci-lint that makes CI runs fail.

#399

Use the official GitHub Action for golangci-lint instead of installing
it manually.
@draftcode draftcode mentioned this pull request Aug 21, 2024
Merged
draftcode added a commit that referenced this pull request Aug 21, 2024
@draftcode
Use the golangci-lint GitHub Action
17f6a8e 
It seems that there's a change in golangci-lint that makes CI runs fail.

#399

Use the official GitHub Action for golangci-lint instead of installing
it manually. This also fixes the lint errors.
draftcode added a commit that referenced this pull request Aug 21, 2024
@draftcode
Use the golangci-lint GitHub Action
2db6c6c 
It seems that there's a change in golangci-lint that makes CI runs fail.

#399

Use the official GitHub Action for golangci-lint instead of installing
it manually.

#402
aviator-app bot pushed a commit that referenced this pull request Aug 21, 2024
@draftcode
Use the golangci-lint GitHub Action (#401)
77bdc9e 
It seems that there's a change in golangci-lint that makes CI runs fail.

#399

Use the official GitHub Action for golangci-lint instead of installing
it manually.

#402
@draftcode draftcode removed the blocked label Aug 21, 2024
@aviator-app aviator-app bot merged commit 477b119 into master Aug 21, 2024
4 checks passed
@aviator-app aviator-app bot deleted the add_demo branch August 21, 2024 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jainankit jainankit jainankit approved these changes

@alex-aviator alex-aviator Awaiting requested review from alex-aviator

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
@draftcode @jainankit