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 warnings from Guides generation #35582

Merged
merged 1 commit into from Mar 12, 2019

Conversation

vipulnsward
Copy link
Member

After #35281 and #35036 AV warns about not having the compiled container.

Use suggested with_empty_template_cache to overcome warning

…led container.

Use the suggested with_empty_template_cache to overcome warning
@rails-bot rails-bot bot added the docs label Mar 12, 2019
@kaspth
Copy link
Contributor

kaspth commented Mar 12, 2019

Where are you seeing these warnings? I can't find them here for instance: https://buildkite.com/rails/rails/builds/59462#563cc025-f4f0-4bc2-a7c8-358ea4441062

@vipulnsward
Copy link
Member Author

I was trying to verify something locally: https://gist.github.com/vipulnsward/41fe9bf0ebf0ac1217699bd64815b638

@vipulnsward
Copy link
Member Author

I see https://buildkite.com/rails/rails/builds/59462#563cc025-f4f0-4bc2-a7c8-358ea4441062 .

We don't test guides generation on CI, it just tests bug report templates.

@kaspth kaspth merged commit a04a757 into rails:master Mar 12, 2019
@kaspth
Copy link
Contributor

kaspth commented Mar 12, 2019

Cool! I had no doubt this was correct, I just wanted to see a before and after :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants