Skip to content

Conversation

@totoprayogo1916
Copy link
Contributor

@totoprayogo1916 totoprayogo1916 commented Dec 23, 2020

Description
Update the echo samples

Checklist:

  • Securely signed commits
  • Component(s) with PHPdocs
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@MGatner
Copy link
Member

MGatner commented Dec 25, 2020

While you're here @totoprayogo1916 would you change the case on booleans? Style Guide specifies they should be "true" and "false".

@totoprayogo1916
Copy link
Contributor Author

While you're here @totoprayogo1916 would you change the case on booleans? Style Guide specifies they should be "true" and "false".

What does it mean?

I just fixed the example on Userguide, and didn't change anything else.

@totoprayogo1916 totoprayogo1916 changed the title UG update the echo example Update examples Dec 25, 2020
@totoprayogo1916 totoprayogo1916 force-pushed the use-short-tag branch 3 times, most recently from 3b10d14 to b18516d Compare December 26, 2020 08:23
Copy link
Member

@MGatner MGatner left a comment

Choose a reason for hiding this comment

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

Nice work. Thank you @totoprayogo1916

@MGatner MGatner merged commit 5a29fcd into codeigniter4:develop Dec 26, 2020
@totoprayogo1916 totoprayogo1916 deleted the use-short-tag branch December 28, 2020 03:43
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.

5 participants