Skip to content

Tools#4

Merged
MGatner merged 5 commits intocodeigniter4:developfrom
MGatner:tools
Aug 13, 2021
Merged

Tools#4
MGatner merged 5 commits intocodeigniter4:developfrom
MGatner:tools

Conversation

@MGatner
Copy link
Copy Markdown
Member

@MGatner MGatner commented Aug 13, 2021

  • Updates the development toolkit
  • Fixes and optimizes test cases
  • Fixes and applies static analysis changes

Note: There are a number of files not PSR-12 compliant, but I have not run them through the style fixer yet to keep this PR clean.

Copy link
Copy Markdown
Member

@lonnieezell lonnieezell left a comment

Choose a reason for hiding this comment

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

I think we have as many files dedicated to third-party tools as we do actual code now. Can't decide if that's good or bad lol.

Comment thread depfile.yaml Outdated
@MGatner
Copy link
Copy Markdown
Member Author

MGatner commented Aug 13, 2021

I know it's a pain, but they end up providing a lot of invaluable automation for the process and can save a lot of redoing things down the road. I work very hard at having a "set-and-forget" workflow in place so it doesn't feel like the tools take over; the setting up part is messy though.

@MGatner
Copy link
Copy Markdown
Member Author

MGatner commented Aug 13, 2021

I just activated this repo in Coveralls - probably takes it a little while to catch up so that failure is not a problem.

@MGatner MGatner merged commit 143391e into codeigniter4:develop Aug 13, 2021
@MGatner MGatner deleted the tools branch August 13, 2021 19:15
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.

2 participants