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

feat: "StarkNet" to "Starknet" #723

Merged

Conversation

0xdorifto
Copy link
Contributor

@0xdorifto 0xdorifto commented Aug 16, 2023

Motivation and Resolution

Because the official name is "Starknet", remove all instances of "StarkNet" in favour of "Starknet".

Issue: #705

Usage related changes

  • All instances of "StarkNet" are now "Starknet"

Development related changes

  • Tests were changed because hashes of strings containing "StarkNet" were changed

Checklist:

  • Performed a self-review of the code
  • Rebased to the last commit of the target branch (or merged it into my branch)
  • Linked the issues which this PR resolves
  • Documented the changes in code (API docs will be generated automatically)
  • Updated the tests
  • All tests are passing

@netlify
Copy link

netlify bot commented Aug 16, 2023

Deploy Preview for starknetjs ready!

Name Link
🔨 Latest commit c11f166
🔍 Latest deploy log https://app.netlify.com/sites/starknetjs/deploys/64dcf9fc16258100081be6c7
😎 Deploy Preview https://deploy-preview-723--starknetjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@0xdorifto 0xdorifto changed the title 0xdorifto stark net to starknet 0xdorifto "StarkNet" to "Starknet" Aug 16, 2023
@0xdorifto 0xdorifto changed the title 0xdorifto "StarkNet" to "Starknet" feat: "StarkNet" to "Starknet" Aug 16, 2023
@ivpavici ivpavici linked an issue Aug 17, 2023 that may be closed by this pull request
@ivpavici
Copy link
Collaborator

@janek26 @dhruvkelawala @yoga-braavos can you please give your input about this? 🙏

@ivpavici
Copy link
Collaborator

ivpavici commented Oct 5, 2023

relevant:
starknet-io/get-starknet#194

Copy link
Collaborator

@tabaktoni tabaktoni left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@penovicp penovicp left a comment

Choose a reason for hiding this comment

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

Leaving a "request changes" so this doesn't get merged yet, I'll prune some of the doc changes beforehand.

Done.

@penovicp penovicp force-pushed the 0xdorifto_StarkNet_to_Starknet branch from c11f166 to 9a2e0e0 Compare October 7, 2023 22:30
Copy link

netlify bot commented Nov 8, 2023

Deploy Preview for starknetjs ready!

Name Link
🔨 Latest commit c9a81bb
🔍 Latest deploy log https://app.netlify.com/sites/starknetjs/deploys/65bca29e6854be000821a763
😎 Deploy Preview https://deploy-preview-723--starknetjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tabaktoni
Copy link
Collaborator

This should be combined with off-chain signing PR

@penovicp penovicp force-pushed the 0xdorifto_StarkNet_to_Starknet branch from 927846e to 800091e Compare February 2, 2024 07:51
@penovicp penovicp force-pushed the 0xdorifto_StarkNet_to_Starknet branch from 800091e to c9a81bb Compare February 2, 2024 08:06
@penovicp penovicp merged commit dd4c5b1 into starknet-io:develop Feb 2, 2024
3 of 4 checks passed
Copy link

github-actions bot commented Feb 2, 2024

🎉 This PR is included in version 6.0.0-beta.14 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

github-actions bot commented Feb 5, 2024

🎉 This PR is included in version 6.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

github-actions bot commented Jul 3, 2024

🎉 This issue has been resolved in version 7.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

casing: Starknet instead of StarkNet
4 participants