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

Fix some typos, reword, correct open source vs source available definition #87

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

lunaynx
Copy link
Contributor

@lunaynx lunaynx commented Sep 27, 2024

Pull request template

Category

  • Add mod to alternatives page
  • Site changes
  • Chore/documentation updates

Describe changes

Why did you do this?

Fixed some typos and incorrect information about open source vs. source available, and reworded some stuff for clarity.

BREAKING CHANGES

  • Tick this box if you made changes that may or may not break site functionality.

Checklist

  • I have clearly stated why I want this to be changed.
  • For major changes, I have clearly stated what they are and why I've done them.
  • Any new resources put in are safe and not malicious.
  • I have searched for duplicate PRs and have not found any.
  • I did not read this section.

Vercel will begin deploying to the Preview environment shortly after submitting this pull request. You may click on the "Visit Preview" link below to see a preview of what your changes look like.
If you updated only part of the repository such that it would not modify the website at all, you do not need to see the Vercel deployment.

You may also use the StackBlitz Codeflow button to quickview the PR in the browser and make changes at the same time if necessary.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello! We've detected that this is your first PR here! If so, thank you! Please make sure everything in your template is correct and that you've reviewed our contributing guidelines, found here: https://notessential.blurry.gay/contributing

Copy link
Member

@Nitrrine Nitrrine left a comment

Choose a reason for hiding this comment

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

LGTM!

@Nitrrine Nitrrine added this pull request to the merge queue Sep 27, 2024
Copy link
Member

@blryface blryface left a comment

Choose a reason for hiding this comment

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

lgtm

@blryface blryface removed this pull request from the merge queue due to a manual request Sep 27, 2024
@blryface
Copy link
Member

I'm not sure what the branch protection does so uh

@blryface blryface added this pull request to the merge queue Sep 27, 2024
@blryface
Copy link
Member

yeah that should do

Merged via the queue into usevital:main with commit 188d327 Sep 27, 2024
2 checks passed
@blryface
Copy link
Member

it's being dumb I'll just force it to merghe

@blryface
Copy link
Member

???? I love internet lag

@blryface
Copy link
Member

I had to remove it from the queue for it to update and merge 🔥

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