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

Remove markdown file update from version increment-cargo-version.sh #27851

Merged
merged 3 commits into from
Sep 19, 2022
Merged

Remove markdown file update from version increment-cargo-version.sh #27851

merged 3 commits into from
Sep 19, 2022

Conversation

willhickey
Copy link
Contributor

Problem

increment-cargo-version.sh increments version numbers in markdown files. This behavior is no longer necessary and sometimes increments version numbers that should be static.

Some discussion here:
https://discord.com/channels/428295358100013066/560503042458517505/1020445231004532810

Summary of Changes

  • Remove markdown file update code
  • Restore version numbers in jsonrpc-api.md

docs/src/developing/clients/jsonrpc-api.md Outdated Show resolved Hide resolved
scripts/increment-cargo-version.sh Show resolved Hide resolved
@@ -1949,7 +1949,7 @@ Returns all accounts owned by the provided program Pubkey
- `offset: <usize>` - offset into program account data to start comparison
- `bytes: <string>` - data to match, as encoded string
- `encoding: <string>` - encoding for filter `bytes` data, either "base58" or "base64". Data is limited in size to 128 or fewer decoded bytes.
**NEW: This field, and base64 support generally, is only available in solana-core v1.15.0 or newer. Please omit when querying nodes on earlier versions**
**NEW: This field, and base64 support generally, is only available in solana-core v1.11.2 or newer. Please omit when querying nodes on earlier versions**
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch!

CriesofCarrots
CriesofCarrots previously approved these changes Sep 17, 2022
Copy link
Contributor

@CriesofCarrots CriesofCarrots left a comment

Choose a reason for hiding this comment

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

r+ reverting the Address Lookup change.
(Also, just for historical documentation purposes, I fixed the Address Lookup versions in v1.14/published docs here: #27806)

@mergify mergify bot dismissed CriesofCarrots’s stale review September 17, 2022 13:12

Pull request has been modified.

@willhickey willhickey merged commit 8f96a39 into solana-labs:master Sep 19, 2022
mergify bot pushed a commit that referenced this pull request Sep 19, 2022
…27851)

* Remove markdown file update from version increment-cargo-version.sh
* Update doc version numbers that were incorrectly advanced by increment-cargo-version.sh
* Revert incorrect doc change based on review feedback

(cherry picked from commit 8f96a39)

# Conflicts:
#	docs/src/developing/clients/jsonrpc-api.md
mergify bot pushed a commit that referenced this pull request Sep 19, 2022
…27851)

* Remove markdown file update from version increment-cargo-version.sh
* Update doc version numbers that were incorrectly advanced by increment-cargo-version.sh
* Revert incorrect doc change based on review feedback

(cherry picked from commit 8f96a39)

# Conflicts:
#	docs/src/developing/clients/jsonrpc-api.md
mergify bot pushed a commit that referenced this pull request Sep 19, 2022
…27851)

* Remove markdown file update from version increment-cargo-version.sh
* Update doc version numbers that were incorrectly advanced by increment-cargo-version.sh
* Revert incorrect doc change based on review feedback

(cherry picked from commit 8f96a39)

# Conflicts:
#	docs/src/developing/clients/jsonrpc-api.md
mergify bot added a commit that referenced this pull request Sep 26, 2022
…backport #27851) (#27928)

* Remove markdown file update from version increment-cargo-version.sh (#27851)

* Remove markdown file update from version increment-cargo-version.sh
* Update doc version numbers that were incorrectly advanced by increment-cargo-version.sh
* Revert incorrect doc change based on review feedback

(cherry picked from commit 8f96a39)

# Conflicts:
#	docs/src/developing/clients/jsonrpc-api.md

* Resolve conflict

Co-authored-by: Will Hickey <[email protected]>
mergify bot added a commit that referenced this pull request Oct 4, 2022
…backport #27851) (#27926)

* Remove markdown file update from version increment-cargo-version.sh (#27851)

* Remove markdown file update from version increment-cargo-version.sh
* Update doc version numbers that were incorrectly advanced by increment-cargo-version.sh
* Revert incorrect doc change based on review feedback

(cherry picked from commit 8f96a39)

# Conflicts:
#	docs/src/developing/clients/jsonrpc-api.md

* Resolve merge conflict

Co-authored-by: Will Hickey <[email protected]>
mergify bot added a commit that referenced this pull request Oct 4, 2022
…backport #27851) (#27927)

* Remove markdown file update from version increment-cargo-version.sh (#27851)

* Remove markdown file update from version increment-cargo-version.sh
* Update doc version numbers that were incorrectly advanced by increment-cargo-version.sh
* Revert incorrect doc change based on review feedback

(cherry picked from commit 8f96a39)

# Conflicts:
#	docs/src/developing/clients/jsonrpc-api.md

* Resolve merge conflict

Co-authored-by: Will Hickey <[email protected]>
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.

2 participants