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

Clarify airnode-examples README for users and devs #1040

Merged
merged 3 commits into from
Apr 28, 2022
Merged

Conversation

dcroote
Copy link
Contributor

@dcroote dcroote commented Apr 27, 2022

  • Removes instructions pertaining to building docker images given we're now using released images
  • Notes that released images are being used during deployment and local client flows
  • Adds instructions for devs to use custom images (e.g. for testing PRs such as api3/airnode-deployer-dev:bb9b8118940ec852c4223b13eba5a6eb97aa3b97 @kolenic-martin )

@dcroote dcroote requested a review from Siegrift April 27, 2022 05:32
@dcroote dcroote self-assigned this Apr 27, 2022
Copy link
Contributor

@Siegrift Siegrift left a comment

Choose a reason for hiding this comment

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

Adds instructions for devs to use custom images (e.g. for testing PRs such as api3/airnode-deployer-dev:bb9b8118940ec852c4223b13eba5a6eb97aa3b97

Is this added somewhere? I do not see it in the README, but I agree it would be nice to add as it's currently only in our private slack channel.

packages/airnode-examples/README.md Outdated Show resolved Hide resolved
@dcroote
Copy link
Contributor Author

dcroote commented Apr 27, 2022

Yes at the end of both new Steps 9 and 10 there is: "If you are a developer and would like to use a different..." I've updated the example commands used there to be -dev images to be clearer.

Copy link
Contributor

@Siegrift Siegrift left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@dcroote dcroote merged commit bb509ca into master Apr 28, 2022
@dcroote dcroote deleted the ex-readme-docker branch April 28, 2022 16: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