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

Replacing validation param #37469

Merged
merged 9 commits into from
Jan 26, 2022
Merged

Conversation

AntonMsk
Copy link
Contributor

In this commit we replaced validation param demo with env

@powerivq
Copy link
Contributor

Hey @AntonMsk please rebase against the latest main branch. It is causing circle failure.

@powerivq
Copy link
Contributor

The PR looks good. Once you can have it rebased and everything passes, I can merge.

@AntonMsk
Copy link
Contributor Author

@powerivq, thanks for fast feedback! Now it should work

@powerivq powerivq merged commit bc18a1d into ampproject:main Jan 26, 2022
rileyajones pushed a commit to rileyajones/amphtml that referenced this pull request Jan 26, 2022
* add myua to vendors list

* fix year and cleaning

* remove copyright notices

* fix linter errors

* replace demo param with env

* replace validation demo param with env
samouri pushed a commit to samouri/amphtml that referenced this pull request Feb 2, 2022
* add myua to vendors list

* fix year and cleaning

* remove copyright notices

* fix linter errors

* replace demo param with env

* replace validation demo param with env
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants