Skip to content

Conversation

@tigarmo
Copy link
Collaborator

@tigarmo tigarmo commented Apr 18, 2023

Some leftover references to the defunct top-level 'env' entry were confusing users; update the example to move the environment definition to the Pebble service. Also fix some minor typos.

  • Have you signed the CLA?

Some leftover references to the defunct top-level 'env' entry were
confusing users; update the example to move the environment definition
to the Pebble service. Also fix some minor typos.
@tigarmo tigarmo requested a review from cjdcordeiro April 18, 2023 13:21
Copy link
Contributor

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

oops. thanks

@tigarmo tigarmo marked this pull request as ready for review April 18, 2023 15:37
@tigarmo tigarmo merged commit c8d27b6 into canonical:main Apr 18, 2023
@tigarmo tigarmo deleted the docs-env-updates branch April 18, 2023 15:38
cjdcordeiro pushed a commit to cjdcordeiro/rockcraft that referenced this pull request Apr 20, 2023
Some leftover references to the defunct top-level 'env' entry were
confusing users; update the example to move the environment definition
to the Pebble service. Also fix some minor typos.
tigarmo pushed a commit that referenced this pull request Mar 24, 2025
Co-authored-by: Callahan <callahan.kovacs@canonical.com>
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.

3 participants