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

fix: update link to upgradeable-contracts #273

Closed

Conversation

0xf333
Copy link
Contributor

@0xf333 0xf333 commented Aug 31, 2023

Outdated links are causing markdown-link-check
check to fail during pull request checks.

Description

Impact

This PR fixes failure associated with upgradeable-contracts link.

Outdated links are causing `markdown-link-check`
check to fail during pull request checks.

Description
===========
- Old link:
  ../paritytech/ink-examples/tree/main/upgradeable-contracts
- Replaced by:
  ../paritytech/ink/tree/master/integration-tests/upgradeable-contracts

Impact
======
This PR fixes failure associated with upgradeable-contracts link.
Copy link
Collaborator

@ascjones ascjones left a comment

Choose a reason for hiding this comment

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

Actually the upgradeable-contracts were removed in use-ink/ink-examples#39, because they were no longer part of ink/integration-tests. However they are now back since use-ink/ink#1889, so instead of changing the links we should restore the upgreadeable-contracts to ink-examples

@0xf333
Copy link
Contributor Author

0xf333 commented Sep 8, 2023

Actually the upgradeable-contracts were removed in paritytech/ink-examples#39, because they were no longer part of ink/integration-tests. However they are now back since paritytech/ink#1889, so instead of changing the links we should restore the upgreadeable-contracts to ink-examples

In that case this PR should be closed then, since the upgradable contract link will remain the same

@0xf333
Copy link
Contributor Author

0xf333 commented Sep 8, 2023

Closing this PR since the upgradable contract link will remain the same.

@0xf333 0xf333 closed this Sep 8, 2023
0xf333 added a commit to 0xf333/ink-examples that referenced this pull request Sep 8, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273
0xf333 added a commit to 0xf333/ink-examples that referenced this pull request Sep 11, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273
0xf333 added a commit to 0xf333/ink-examples that referenced this pull request Sep 11, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273

* fix: update set-code-hash path in ci
0xf333 added a commit to 0xf333/ink-examples that referenced this pull request Sep 11, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273

* fix: update set-code-hash path in ci

* fix: adjust ci for upgradable-contracts directory

Description
===========
- Added environment variable UPGRADEABLE_CONTRACTS.
- Adjusted ci to process contracts under the
  upgradeable-contracts directory.
- Removed unnecessary foo, bar feature flags from the
  CI steps as they were not present in Cargo.toml files
0xf333 added a commit to 0xf333/ink-examples that referenced this pull request Sep 11, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273

* fix: update set-code-hash path in ci

* fix: adjust ci for upgradable-contracts directory

Description
===========
- Added environment variable UPGRADEABLE_CONTRACTS.
- Adjusted ci to process contracts under the
  upgradeable-contracts directory.
- Removed unnecessary foo, bar feature flags from the
  CI steps as they were not present in Cargo.toml files
0xf333 added a commit to 0xf333/ink-examples that referenced this pull request Sep 11, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273

* fix: update set-code-hash path in ci

* fix: adjust ci for upgradable-contracts directory

Description
===========
- Added environment variable UPGRADEABLE_CONTRACTS.
- Adjusted ci to process contracts under the
  upgradeable-contracts directory.
- Removed unnecessary foo, bar feature flags from the
  CI steps as they were not present in Cargo.toml files
ascjones pushed a commit to use-ink/ink-examples that referenced this pull request Sep 12, 2023
Description
===========
This PR serves two purposes:
    - Adjusts ink-examples to align with integration
      test repositories.
    - Fixes the previous URL check failure from Ink-docs.
      For more context, please refer to Ink-docs PR 273.

Impact
======
This PR addresses markdown-link-check issue in Ink-docs:
use-ink/ink-docs#273

* fix: update set-code-hash path in ci

* fix: adjust ci for upgradable-contracts directory

Description
===========
- Added environment variable UPGRADEABLE_CONTRACTS.
- Adjusted ci to process contracts under the
  upgradeable-contracts directory.
- Removed unnecessary foo, bar feature flags from the
  CI steps as they were not present in Cargo.toml files
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