Skip to content

Conversation

ADKaster
Copy link
Member

The contributing docs are lightly edited from the original file in the
ladybird repo. The script is a straight copy paste.

@ADKaster ADKaster requested review from awesomekling and gmta July 15, 2025 22:23
if: ${{ failure() && !github.event.pull_request.draft }}
uses: IdanHo/comment-on-pr@63ea2bf352997c66e524b8b5be7a79163fb3a88a
env:
GITHUB_TOKEN: ${{ secrets.LADYBIRD_BOT_TOKEN }}
Copy link
Member Author

Choose a reason for hiding this comment

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

same Q for the bot token. org-wide, or needs owner fiddling in the settings?

Copy link
Contributor

@gmta gmta Jul 24, 2025

Choose a reason for hiding this comment

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

I think we should move LADYBIRD_BOT_TOKEN to org-wide secrets, it currently lives in the ladybird repo.


Ping them right away if it's something urgent! If it's less urgent, advertise your PR on Discord (`#code-review`) and ask if someone could review it.

### Is Discord the place to ask for review help, or is GitHub preferred?
Copy link
Member Author

Choose a reason for hiding this comment

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

the note on stalebot was deleted here, but can be added back if we turn it on for this repo.

ADKaster added 2 commits July 15, 2025 16:28
The contributing docs are lightly edited from the original file in the
ladybird repo. The script is a straight copy paste.
This is a straight copy from the ladybird repo, with a single docs link
changed in the bot message.
@ADKaster ADKaster force-pushed the contributing-guidelinez branch from 49c804f to fb3de79 Compare July 15, 2025 22:28
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.

2 participants