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

docs(removal): remove old packages #4059

Merged
merged 1 commit into from
Aug 11, 2023

Conversation

fl-y
Copy link
Contributor

@fl-y fl-y commented Aug 11, 2023

@JafarAz

Required for merge:

pr-workflow-check / draft-release-check is ✅ success
Other rules GitHub shows you, or can be read in configuration
Makes review faster:

PR title is my best effort to provide summary of changes and has clear text to be part of release notes
I marked PR by misc label if it should not be in release notes
Linked Zenhub/Github/Slack/etc reference if one exists
I was clear on what type of deployment required to release my changes (node, runtime, contract, indexer, on chain operation, frontend, infrastructure) if any in PR title or description
Added reviewer into Reviewers
I tagged(@) or used other form of notification of one person who I think can handle best review of this PR
I have proved that PR has no general regressions of relevant features and processes required to release into production
Any dependency updates made, was done according guides from relevant dependency
Clicking all checkboxes
Adding detailed description of changes when it feels appropriate (for example when PR is big)

@fl-y fl-y requested a review from JafarAz August 11, 2023 12:13
@JafarAz JafarAz enabled auto-merge August 11, 2023 18:26
@JafarAz JafarAz added this pull request to the merge queue Aug 11, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 11, 2023
@dzmitry-lahoda
Copy link
Contributor

retry please as per ci issue template

@dzmitry-lahoda
Copy link
Contributor

ci is distributed system and sometimes need to retry

@JafarAz
Copy link
Collaborator

JafarAz commented Aug 11, 2023

retry pr or ci? I re try failed job twice now

@dzmitry-lahoda
Copy link
Contributor

@JafarAz would you press Merge When Ready?

@JafarAz JafarAz added this pull request to the merge queue Aug 11, 2023
Merged via the queue into ComposableFi:main with commit 91d4515 Aug 11, 2023
66 of 68 checks passed
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.

3 participants