Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Gradual Dutch Auction Module #177

Open
wants to merge 71 commits into
base: develop
Choose a base branch
from
Open

Gradual Dutch Auction Module #177

wants to merge 71 commits into from

Conversation

Oighty
Copy link
Contributor

@Oighty Oighty commented May 1, 2024

Clean implementation based on new priorities for #1 and the updated codebase.

Tasks:

  • Implement core features
  • Address outstanding TODOs
  • Write tests

install.sh Outdated
@@ -21,7 +21,7 @@ cd lib/forge-std/ && git checkout v1.7.1 && cd ../..

echo ""
echo "prb-math"
cd lib/prb-math/ && git checkout v4.0.2 && cd ../..
cd lib/prb-math/ && git checkout main && cd ../..
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest pinning this to a commit eventually

Copy link
Contributor Author

@Oighty Oighty May 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I plan to. I just set it to main incase I needed to push any more changes to the library. Will leave this convo open as a reminder.

@Oighty Oighty changed the base branch from develop to fixed-batch May 22, 2024 16:01
Base automatically changed from fixed-batch to develop June 6, 2024 13:26
@Oighty Oighty changed the title [DRAFT] Gradual Dutch Auction Module Gradual Dutch Auction Module Sep 19, 2024
@Oighty Oighty requested a review from 0xJem September 19, 2024 20:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants