feat: update suzuka #2326
Merged
feat: update suzuka #2326
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This reverts commit 0e9ab96.
cdummett
force-pushed
the
feat/update_suzuka
branch
4 times, most recently
from
1cd7d79 to
c8ba689
Compare
cdummett
force-pushed
the
feat/update_suzuka
branch
from
c8ba689 to
9a1a3c3
Compare
* feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]>
feat: calculate fraction of cumulative taker volume across window
davidsiska-vega
approved these changes
Jul 31, 2024
cdummett
added a commit
that referenced
this pull request
Jul 31, 2024
* feat: add AC for vAMM during opening auction * chore: typo * fix: reword * feat: Updates to fix up equations (#2303) * feat: tx trade ordering (#2231) * feat: Order trade transactions within a block * feat: AMM Estimates (#2282) * feat: Estimate within range * refactor: first stab at max block auction * refactor: add defaults * refactor: formalise action interaction + ACs * refactor: remove supplied liquidity requirement from ACs * chore: update approbation * chore: fix CI issues * milestone rejig * fix fee mechanic milestone, reorder features * chore: remove impossible AC, but mention the behaviour in spec Signed-off-by: Elias Van Ootegem <[email protected]> * chore: remove old AC from json Signed-off-by: Elias Van Ootegem <[email protected]> * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * rename 3 and rejig milestones * sorting * feat: change to average notional position * feat: calculate fraction of cumulative volume across window rather than average of fractions across window * fix: update spellcheck * Order spam (#2188) * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: wording Co-authored-by: Charlie <[email protected]> * fix: spot correction Co-authored-by: Charlie <[email protected]> * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * chore: clairfy spam protection for spot markets * fix: remove * fix: don't allow non-persistent orders that don't meet margin / value criteria * feat: Update spam tests on amendments * feat: Update spam tests on amendments * chore: revert inconsistent formatting * chore: fix markdownlint --------- Co-authored-by: Charlie <[email protected]> Co-authored-by: Charlie <[email protected]> Co-authored-by: Tom McLean <[email protected]> * Revert "feat: average notional position" (#2320) * feat: update suzuka (#2326) * Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> * chore: update feature files --------- Signed-off-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: David Siska <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Edd <[email protected]> Co-authored-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jeremy Letang <[email protected]>
cdummett
added a commit
that referenced
this pull request
Sep 16, 2024
* Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]>
cdummett
added a commit
that referenced
this pull request
Sep 17, 2024
* feat: add AC for vAMM during opening auction * chore: typo * fix: reword * feat: Updates to fix up equations (#2303) * feat: tx trade ordering (#2231) * feat: Order trade transactions within a block * feat: AMM Estimates (#2282) * feat: Estimate within range * refactor: first stab at max block auction * refactor: add defaults * refactor: formalise action interaction + ACs * refactor: remove supplied liquidity requirement from ACs * chore: update approbation * chore: fix CI issues * milestone rejig * fix fee mechanic milestone, reorder features * chore: remove impossible AC, but mention the behaviour in spec Signed-off-by: Elias Van Ootegem <[email protected]> * chore: remove old AC from json Signed-off-by: Elias Van Ootegem <[email protected]> * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * rename 3 and rejig milestones * sorting * feat: change to average notional position * feat: calculate fraction of cumulative volume across window rather than average of fractions across window * fix: update spellcheck * Order spam (#2188) * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: wording Co-authored-by: Charlie <[email protected]> * fix: spot correction Co-authored-by: Charlie <[email protected]> * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * chore: clairfy spam protection for spot markets * fix: remove * fix: don't allow non-persistent orders that don't meet margin / value criteria * feat: Update spam tests on amendments * feat: Update spam tests on amendments * chore: revert inconsistent formatting * chore: fix markdownlint --------- Co-authored-by: Charlie <[email protected]> Co-authored-by: Charlie <[email protected]> Co-authored-by: Tom McLean <[email protected]> * Revert "feat: average notional position" (#2320) * feat: update suzuka (#2326) * Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> * chore: update feature files --------- Signed-off-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: David Siska <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Edd <[email protected]> Co-authored-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jeremy Letang <[email protected]>
cdummett
added a commit
that referenced
this pull request
Sep 17, 2024
* feat: add AC for vAMM during opening auction * chore: typo * fix: reword * feat: Updates to fix up equations (#2303) * feat: tx trade ordering (#2231) * feat: Order trade transactions within a block * feat: AMM Estimates (#2282) * feat: Estimate within range * refactor: first stab at max block auction * refactor: add defaults * refactor: formalise action interaction + ACs * refactor: remove supplied liquidity requirement from ACs * chore: update approbation * chore: fix CI issues * milestone rejig * fix fee mechanic milestone, reorder features * chore: remove impossible AC, but mention the behaviour in spec Signed-off-by: Elias Van Ootegem <[email protected]> * chore: remove old AC from json Signed-off-by: Elias Van Ootegem <[email protected]> * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * rename 3 and rejig milestones * sorting * feat: change to average notional position * feat: calculate fraction of cumulative volume across window rather than average of fractions across window * fix: update spellcheck * Order spam (#2188) * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: wording Co-authored-by: Charlie <[email protected]> * fix: spot correction Co-authored-by: Charlie <[email protected]> * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * chore: clairfy spam protection for spot markets * fix: remove * fix: don't allow non-persistent orders that don't meet margin / value criteria * feat: Update spam tests on amendments * feat: Update spam tests on amendments * chore: revert inconsistent formatting * chore: fix markdownlint --------- Co-authored-by: Charlie <[email protected]> Co-authored-by: Charlie <[email protected]> Co-authored-by: Tom McLean <[email protected]> * Revert "feat: average notional position" (#2320) * feat: update suzuka (#2326) * Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> * chore: update feature files --------- Signed-off-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: David Siska <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Edd <[email protected]> Co-authored-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jeremy Letang <[email protected]>
cdummett
added a commit
that referenced
this pull request
Sep 17, 2024
* feat: add AC for vAMM during opening auction * chore: typo * fix: reword * feat: Updates to fix up equations (#2303) * feat: tx trade ordering (#2231) * feat: Order trade transactions within a block * feat: AMM Estimates (#2282) * feat: Estimate within range * refactor: first stab at max block auction * refactor: add defaults * refactor: formalise action interaction + ACs * refactor: remove supplied liquidity requirement from ACs * chore: update approbation * chore: fix CI issues * milestone rejig * fix fee mechanic milestone, reorder features * chore: remove impossible AC, but mention the behaviour in spec Signed-off-by: Elias Van Ootegem <[email protected]> * chore: remove old AC from json Signed-off-by: Elias Van Ootegem <[email protected]> * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * rename 3 and rejig milestones * sorting * feat: change to average notional position * feat: calculate fraction of cumulative volume across window rather than average of fractions across window * fix: update spellcheck * Order spam (#2188) * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: wording Co-authored-by: Charlie <[email protected]> * fix: spot correction Co-authored-by: Charlie <[email protected]> * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * chore: clairfy spam protection for spot markets * fix: remove * fix: don't allow non-persistent orders that don't meet margin / value criteria * feat: Update spam tests on amendments * feat: Update spam tests on amendments * chore: revert inconsistent formatting * chore: fix markdownlint --------- Co-authored-by: Charlie <[email protected]> Co-authored-by: Charlie <[email protected]> Co-authored-by: Tom McLean <[email protected]> * Revert "feat: average notional position" (#2320) * feat: update suzuka (#2326) * Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> * chore: update feature files --------- Signed-off-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: David Siska <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Edd <[email protected]> Co-authored-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jeremy Letang <[email protected]>
cdummett
added a commit
that referenced
this pull request
Sep 17, 2024
* feat: add AC for vAMM during opening auction * chore: typo * fix: reword * feat: Updates to fix up equations (#2303) * feat: tx trade ordering (#2231) * feat: Order trade transactions within a block * feat: AMM Estimates (#2282) * feat: Estimate within range * refactor: first stab at max block auction * refactor: add defaults * refactor: formalise action interaction + ACs * refactor: remove supplied liquidity requirement from ACs * chore: update approbation * chore: fix CI issues * milestone rejig * fix fee mechanic milestone, reorder features * chore: remove impossible AC, but mention the behaviour in spec Signed-off-by: Elias Van Ootegem <[email protected]> * chore: remove old AC from json Signed-off-by: Elias Van Ootegem <[email protected]> * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * rename 3 and rejig milestones * sorting * feat: change to average notional position * feat: calculate fraction of cumulative volume across window rather than average of fractions across window * fix: update spellcheck * Order spam (#2188) * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: wording Co-authored-by: Charlie <[email protected]> * fix: spot correction Co-authored-by: Charlie <[email protected]> * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * chore: clairfy spam protection for spot markets * fix: remove * fix: don't allow non-persistent orders that don't meet margin / value criteria * feat: Update spam tests on amendments * feat: Update spam tests on amendments * chore: revert inconsistent formatting * chore: fix markdownlint --------- Co-authored-by: Charlie <[email protected]> Co-authored-by: Charlie <[email protected]> Co-authored-by: Tom McLean <[email protected]> * Revert "feat: average notional position" (#2320) * feat: update suzuka (#2326) * Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> * chore: update feature files --------- Signed-off-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: David Siska <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Edd <[email protected]> Co-authored-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jeremy Letang <[email protected]>
cdummett
added a commit
that referenced
this pull request
Sep 17, 2024
* feat: add AC for vAMM during opening auction * chore: typo * fix: reword * feat: Updates to fix up equations (#2303) * feat: tx trade ordering (#2231) * feat: Order trade transactions within a block * feat: AMM Estimates (#2282) * feat: Estimate within range * refactor: first stab at max block auction * refactor: add defaults * refactor: formalise action interaction + ACs * refactor: remove supplied liquidity requirement from ACs * chore: update approbation * chore: fix CI issues * milestone rejig * fix fee mechanic milestone, reorder features * chore: remove impossible AC, but mention the behaviour in spec Signed-off-by: Elias Van Ootegem <[email protected]> * chore: remove old AC from json Signed-off-by: Elias Van Ootegem <[email protected]> * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * rename 3 and rejig milestones * sorting * feat: change to average notional position * feat: calculate fraction of cumulative volume across window rather than average of fractions across window * fix: update spellcheck * Order spam (#2188) * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: wording Co-authored-by: Charlie <[email protected]> * fix: spot correction Co-authored-by: Charlie <[email protected]> * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * fix: clarify order spam * chore: clairfy spam protection for spot markets * fix: remove * fix: don't allow non-persistent orders that don't meet margin / value criteria * feat: Update spam tests on amendments * feat: Update spam tests on amendments * chore: revert inconsistent formatting * chore: fix markdownlint --------- Co-authored-by: Charlie <[email protected]> Co-authored-by: Charlie <[email protected]> Co-authored-by: Tom McLean <[email protected]> * Revert "feat: average notional position" (#2320) * feat: update suzuka (#2326) * Revert "Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update" This reverts commit 09dd85a, reversing changes made to 44ecde7. * Revert "feat: separate fee discount components (#2293)" This reverts commit 0e9ab96. * chore: trigger checks * feat: separate fee discount components (#2293) * feat: Adding separate discount factor components * feat: Adding separate discount factor components * feat: Add ACs * feat: Fix AC code * feat: Fix AC code * feat: Review comments * feat: Adding high volume maker rebate * feat: Fix up eqs * feat: Remove duplicative naming * chore: Fix up markdown * refactor: add default value --------- Co-authored-by: Witold <[email protected]> * Merge pull request #2315 from vegaprotocol/feat/maker_rebate_update feat: calculate fraction of cumulative taker volume across window --------- Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> * chore: update feature files --------- Signed-off-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: David Siska <[email protected]> Co-authored-by: Jiajia-Cui <[email protected]> Co-authored-by: Tom <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Witold <[email protected]> Co-authored-by: Edd <[email protected]> Co-authored-by: Elias Van Ootegem <[email protected]> Co-authored-by: Jeremy Letang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rebases on Colosseo II then cherry-picks the reverted commits.