Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Adds GitHub setup files #120

Merged
merged 5 commits into from
Apr 24, 2024
Merged

Adds GitHub setup files #120

merged 5 commits into from
Apr 24, 2024

Conversation

arthurgousset
Copy link
Contributor

@arthurgousset arthurgousset commented Mar 23, 2024 •
edited
Loading

Description

  • Changes LICENSE from MIT to Apache 2.0. This is consistent with the other packages in celo-org/developer-tooling
  • Adds bug report and feature request template
  • Adds CONTRIBUTING.md file

Other changes

None

Tested

Only docs changes.

Related issues

Backwards compatibility

Yes

Documentation

Yes

@arthurgousset arthurgousset requested a review from a team as a code owner March 23, 2024 10:44
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Mar 23, 2024 •
edited
Loading

⚠️ No Changeset found

Latest commit: 6a51816

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel Vercel
Copy link

vercel bot commented Mar 23, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rainbowkit-with-celo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2024 9:32am

@arthurgousset arthurgousset marked this pull request as draft March 23, 2024 10:45
@arthurgousset arthurgousset self-assigned this Mar 23, 2024
@arthurgousset
Copy link
Contributor Author

Blocked by

Once all wallets are in the upstream repo and a simple migration guide exists, I'll update the README in this PR and merge it.

@arthurgousset
Copy link
Contributor Author

Maybe I should take the README commit out of this PR, so it can be merged and open a draft README PR for when we are ready to move forward with deprecating the package.

@arthurgousset arthurgousset force-pushed the arthurgousset/github-setup branch from 36a207b to e541501 Compare April 24, 2024 09:13
@arthurgousset arthurgousset force-pushed the arthurgousset/github-setup branch from e541501 to 6a51816 Compare April 24, 2024 09:31
@arthurgousset
Copy link
Contributor Author

arthurgousset commented Apr 24, 2024 •
edited
Loading

Maybe I should take the README commit out of this PR, so it can be merged and open a draft README PR for when we are ready to move forward with deprecating the package.

That's done ✅ I moved the incomplete disclaimer (in the README.md file) that this package might be deprecated in the into a separate PR:

That should allow these changes to be merged without being blocked by the deprecation process.

@arthurgousset arthurgousset marked this pull request as ready for review April 24, 2024 09:54
@arthurgousset arthurgousset requested review from a team and removed request for a team April 24, 2024 09:55
@arthurgousset arthurgousset merged commit d387765 into main Apr 24, 2024
6 checks passed
@arthurgousset arthurgousset deleted the arthurgousset/github-setup branch April 24, 2024 10:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@aaronmgdr aaronmgdr aaronmgdr approved these changes

Assignees

@arthurgousset arthurgousset

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update GitHub setup files
2 participants
@arthurgousset @aaronmgdr