Skip to content

Conversation

@erinecon
Copy link
Contributor

@erinecon erinecon commented Feb 24, 2025

Maintenance work on tutorials that spans across the setup scripts, the tutorial index page and all four tutorials.

Summary of work

  • Renamed the files so that the URLs will be shortened
  • In the setup files, updated some code-block directives to use text to fix weird highlighting issues in the render
  • Update the "What you'll do" section to fix the tone and "collapsed list" feel
  • Changes to microcopy/terminology patterns to establish consistency with industry standards, accessibility, and Starcraft terminology

erinecon and others added 5 commits February 24, 2025 14:48
- Fix indentation
- Remove parentheses
- Fix two lines before headings
- Correct use of "architecture"
- amd64 → AMD64
- arm64 → ARM64
- application → app
- Use common contractions
- Remove unnecessary colons
- ’ → '
@erinecon erinecon marked this pull request as ready for review February 25, 2025 12:48
Copy link
Contributor

@medubelko medubelko left a comment

Choose a reason for hiding this comment

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

LGTM!

to test, deploy, configure, etc., your Django application on a local
Kubernetes cloud, ``microk8s``, with ``juju``. All of that multiple
times, mimicking a real development process.
- Create a Django app.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe these should be ordered?

Copy link
Collaborator

@lengau lengau left a comment

Choose a reason for hiding this comment

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

LGTM! One question, but it's addressable separately.

@lengau lengau merged commit ecd6b41 into canonical:main Feb 26, 2025
22 of 31 checks passed
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