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: reintroduce upgradable contract examples #41

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

0xf333
Copy link
Contributor

@0xf333 0xf333 commented Sep 8, 2023

Hi @ascjones
This is about the previous chat regarding the Ink-docs markdown-link-check failure during PRs.
Now the link to the upgradable contract will be operational.

Description

This PR serves two purposes:

  • Adjusts ink-examples to align with integration test repos.
  • Fixes the previous URL check failure from Ink-docs.
  • Adjusts ci to process contracts under the upgradeable-contracts directory.
  • Removes unnecessary foo, bar feature flags from the CI steps as they are not present in Cargo.toml files

Impact

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

.

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.

Thanks!

@ascjones
Copy link
Collaborator

@0xf333
Copy link
Contributor Author

0xf333 commented Sep 11, 2023

CI is broken, see https://github.com/paritytech/ink-examples/actions/runs/6145166786/job/16672011933?pr=41#step:13:512

Fixed,
Just waiting for ci checks to complete now.

@0xf333 0xf333 force-pushed the 0xf333/upgradable_contracts branch 2 times, most recently from f828978 to b000ffa Compare September 11, 2023 16:22
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
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.

LGTM

@ascjones ascjones merged commit b0e00fb into use-ink:main Sep 12, 2023
5 checks passed
@0xf333 0xf333 deleted the 0xf333/upgradable_contracts branch May 9, 2024 08:21
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