-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Init Changelog Sync Script #5776
Conversation
…Update format NPM script to format .ts files in scripts dir
…o wyatt/4.x/5451-refactor
### Added | ||
|
||
#### web3-core | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bool | ||
) | ||
{ | ||
function getValues() public view returns (uint256, string memory, bool) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These file changes are a result of running yarn format
@@ -3,29 +3,29 @@ | |||
pragma solidity ^0.8.13; | |||
|
|||
contract Greeter { | |||
uint256 counter; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These file changes are a result of running yarn format
id, | ||
sub, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This error was reported after running yarn lint:fix
@@ -18,17 +18,16 @@ along with web3.js. If not, see <http://www.gnu.org/licenses/>. | |||
pragma solidity ^0.8.7; | |||
|
|||
contract SampleStorageContract { | |||
uint256 uintNum; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These file changes are a result of running yarn format
@@ -28,7 +28,7 @@ import { isAbiParameterSchema } from './validation/abi'; | |||
import { isHexStrict } from './validation/string'; | |||
// import { abiToJsonSchemaCases } from '../test/fixtures/abi_to_json_schema'; | |||
|
|||
export const parseBaseType = <T = typeof VALID_ETH_BASE_TYPES[number]>( | |||
export const parseBaseType = <T = (typeof VALID_ETH_BASE_TYPES)[number]>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These file changes are a result of running yarn format
Deploying with Cloudflare Pages
|
Bundle StatsHey there, this message comes from a github action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger No assets were bigger Smaller No assets were smaller Unchanged
|
Great work @spacesailor24 , |
Supersedes #5769 and closes #5451
What This PR Does
This PR introduces the script-pseudo-package
changelog
(found underscripts/changelog
). It currently supports the following commands:sync
: Takes the changes from the## [Unreleased]
sections of each package'sCHANGELOG.md
and adds it to the rootCHANGELOG.md
The following commands will add an entry to a specific package's
CHANGELOG.md
under the provided header:added
changed
deprecated
removed
fixed
security
Any other command will display the help table;
Usage
To use the script, in the project root, run
yarn changelog <command> <arguments>
e.g.:yarn changelog sync
yarn changelog added web3 "I added this change (#1234)"
yarn changelog removed web3 "I removed something (#1235)"