Skip to content

feat: init reviewing-prs#100

Open
orangci wants to merge 3 commits intomainfrom
draft-reviewing-prs
Open

feat: init reviewing-prs#100
orangci wants to merge 3 commits intomainfrom
draft-reviewing-prs

Conversation

@orangci
Copy link
Contributor

@orangci orangci commented Aug 19, 2025

A document to ease and guide new (and old) maintainers into understanding the pull request reviewing process in is-a.dev.

@orangci
Copy link
Contributor Author

orangci commented Mar 4, 2026

wait what the hell is this when did I do this

@orangci orangci marked this pull request as ready for review March 4, 2026 20:34
@orangci orangci requested a review from a team as a code owner March 4, 2026 20:34
@DEV-DIBSTER
Copy link
Member

wait what the hell is this when did I do this

Idk bruh you were supposed to finish this awhile ago or something.

Copy link
Member

@iostpa iostpa left a comment

Choose a reason for hiding this comment

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

The approve checks section may need to be rewritten since most PRs are ran by default, it will only be required to approve checks if the GitHub account is new

1. The website has no preview provided *and* isn't accessible. We expect previews for *all* webpages.
2. The checkbox stating the user's agreement to our ToS has not been checked.
3. The PR is requesting NS records without a valid reason or without a reason provided at all. See [this page](https://docs.is-a.dev/faq/#who-can-use-ns-records) for details on what we consider a valid reason to request NS records.
4. The user violates our ToS in any way; e.g., if they are under thirteen years of age, if their webpage is breaking the law or hosting illegal content, cryptocurrency mining sites or tools, mass unsolicited emailing, impersonation without consent, et cetera.
Copy link
Member

Choose a reason for hiding this comment

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

I haven't seen the unsolicited emailing on the ToS recently so not sure if it should be removed from here

@iostpa
Copy link
Member

iostpa commented Mar 5, 2026

wait what the hell is this when did I do this

Idk bruh you were supposed to finish this awhile ago or something.

A month or so after the PR was made he left the maintainer team and this PR was just existing

orangci and others added 2 commits March 5, 2026 17:42
Co-authored-by: iostpa <iostpa@iostpa.com>
Co-authored-by: iostpa <iostpa@iostpa.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants