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

Add config validation for local Airnode invocation including airnode-client container #1105

Merged
merged 1 commit into from
May 20, 2022

Conversation

amarthadan
Copy link
Contributor

AN-595

Doing the config file validation separately (instead of running it as a part of invoke.ts) so that if it fails, the container fails as well instead of running and throwing an error every minute.

@amarthadan amarthadan requested review from Siegrift and a team May 19, 2022 11:31
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.

Currently the tests fail.

packages/airnode-node/src/cli/validate-config.ts Outdated Show resolved Hide resolved
packages/airnode-examples/scripts/run-airnode-locally.ts Outdated Show resolved Hide resolved
packages/airnode-node/package.json Show resolved Hide resolved
@amarthadan amarthadan force-pushed the validator-airnode-client branch 2 times, most recently from 49d7bc5 to 5f70853 Compare May 19, 2022 13:46
@amarthadan amarthadan requested review from Siegrift and a team May 19, 2022 14:01
packages/airnode-examples/README.md Show resolved Hide resolved
@@ -28,6 +28,7 @@
"@api3/airnode-protocol": "^0.6.0",
"@api3/airnode-utilities": "^0.6.0",
"@api3/airnode-validator": "^0.6.0",
"@api3/promise-utils": "^0.3.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we using it somewhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I initially was but ended up not using it and forget to remove it. Will do.

@amarthadan amarthadan force-pushed the validator-airnode-client branch from 5f70853 to e4c1a22 Compare May 20, 2022 08:27
@amarthadan amarthadan requested a review from Siegrift May 20, 2022 11:49
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

@amarthadan amarthadan merged commit 620b35b into master May 20, 2022
@amarthadan amarthadan deleted the validator-airnode-client branch May 20, 2022 12: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