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

✨ add notes history for Promise.withResolvers() #508

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

ctcpip
Copy link
Member

@ctcpip ctcpip commented Feb 14, 2024

No description provided.

finished-proposals.md Outdated Show resolved Hide resolved
@ljharb
Copy link
Member

ljharb commented Feb 14, 2024

Looks pretty good! altho at this point it seems better to use the <br />s than the commas - on my screen it's always one per line anyways.

@ctcpip
Copy link
Member Author

ctcpip commented Feb 14, 2024

two lines for me

Screenshot 2024-02-14 at 5 20 49 PM

edit: no, I am not zoomed out 😅

@ljharb
Copy link
Member

ljharb commented Feb 14, 2024

very strange, keep the commas then.

@ctcpip ctcpip force-pushed the history-example branch 2 times, most recently from bce81cd to 06c5f39 Compare February 15, 2024 07:48
finished-proposals.md Outdated Show resolved Hide resolved
@ljharb ljharb force-pushed the history-example branch 5 times, most recently from c4faec2 to e8bd86c Compare September 12, 2024 06:01
@ljharb ljharb marked this pull request as ready for review September 12, 2024 06:01
Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

k i tweaked it a bit and i'm happy with it now; i'm going to land this, and would love further PRs to flesh out the notes history for all the proposals (in all the tables)

@ljharb ljharb merged commit e8bd86c into main Sep 12, 2024
@ljharb ljharb deleted the history-example branch September 12, 2024 06:02
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