Skip to content

update readme #880

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.

Sign up for GitHub

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

Jump to bottom
Merged
felicio merged 3 commits into from
Dec 17, 2025
Merged

update readme #880

felicio merged 3 commits into from
Dec 17, 2025

Conversation

@felicio
Copy link
Collaborator

@felicio felicio commented Dec 10, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 10, 2025 07:20
@changeset-bot
Copy link

changeset-bot bot commented Dec 10, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: e48d2b1

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
Copy link

vercel bot commented Dec 10, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
status-api Ready Ready Preview, Comment Dec 17, 2025 10:04am
status-components Ready Ready Preview, Comment Dec 17, 2025 10:04am
status-network-hub Ready Ready Preview, Comment Dec 17, 2025 10:04am
status-network-website Ready Ready Preview, Comment Dec 17, 2025 10:04am
status-portfolio Ready Ready Preview, Comment Dec 17, 2025 10:04am
status-website Ready Ready Preview, Comment Dec 17, 2025 10:04am

@felicio felicio requested review from a team, JulesFiliot, jinhojang6 and jkbktl December 10, 2025 07:21
Copilot AI reviewed Dec 10, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reorganizes the README to streamline the development setup instructions and add documentation for the publishing process. The changes consolidate Nix-related information and introduce a new Publishing section.

  • Nix moved from a dedicated section to an optional prerequisite with inline setup instructions
  • Added a new Publishing section explaining the automated release process
  • Simplified the development mode section by removing duplicate Nix setup instructions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md Outdated
## Publishing
To publish packages to npm registry merge PR named Release, which is automatically created and updated by Chnageset on merge of any other PR containing changesets files.
Copy link

Copilot AI Dec 10, 2025

Choose a reason for hiding this comment

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

The sentence structure is unclear and contains grammatical issues. Consider rephrasing to: "To publish packages to the npm registry, merge the PR named 'Release', which is automatically created and updated by Changeset when any other PR containing changeset files is merged."

Suggested change
To publish packages to npm registry merge PR named Release, which is automatically created and updated by Chnageset on merge of any other PR containing changesets files.
To publish packages to the npm registry, merge the PR named 'Release', which is automatically created and updated by Changeset when any other PR containing changeset files is merged.

Copilot uses AI. Check for mistakes.
JulesFiliot
JulesFiliot requested changes Dec 10, 2025
View reviewed changes
Copy link
Contributor

@JulesFiliot JulesFiliot left a comment

Choose a reason for hiding this comment

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

🙏

@github-project-automation github-project-automation bot moved this from In Progress to ToDo in Web & User Interfaces Dec 10, 2025
@felicio @JulesFiliot
Update README.md
985925e 
Co-authored-by: Jules <[email protected]>
JulesFiliot
JulesFiliot approved these changes Dec 10, 2025
View reviewed changes
Copy link
Contributor

@JulesFiliot JulesFiliot left a comment

Choose a reason for hiding this comment

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

:shipit:

@github-project-automation github-project-automation bot moved this from ToDo to Waiting in Web & User Interfaces Dec 10, 2025
@status-im-auto
Copy link
Member

status-im-auto commented Dec 17, 2025 •
edited
Loading

Jenkins Builds

❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ e48d2b1 #3 2025-12-17 07:43:23 ~41 sec connector 📦zip
✔️ e48d2b1 #3 2025-12-17 07:43:48 ~1 min wallet 📦zip

@felicio felicio merged commit 8b7c83b into main Dec 17, 2025
13 checks passed
@felicio felicio deleted the felicio/u/readme-publish branch December 17, 2025 10:22
@github-project-automation github-project-automation bot moved this from Waiting to Done in Web & User Interfaces Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@jkbktl jkbktl jkbktl approved these changes

@jinhojang6 jinhojang6 jinhojang6 approved these changes

@JulesFiliot JulesFiliot JulesFiliot approved these changes

Assignees

No one assigned

Labels

None yet

Projects

Web & User Interfaces
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@felicio @status-im-auto @jkbktl @jinhojang6 @JulesFiliot