Developer Area/Developer Meetings/93
From Mahara Wiki
Agenda for the 93rd Mahara developer meeting on Tuesday, 24 January 2023 at 23:00 UTC
We will meet online using Big Blue Button (A Catalyst staff member will initiate the call).
Our #Mahara channel on Matrix will be our backup in case there are problems with the web conferencing tool and we'll need to chat to resolve it. You can connect to our Matrix channel also using the #mahara channel on Freenode IRC.
- Chair: Doris Tam
- Minute taker: Gold
Items from previous meetings
- Change to makefile to add some basic security checks
- Update coding and code review guidelines to add words around security
- PHP 8.1 progress
- Idea to progress towards PHP 8.1: Introduce a project-wide linter
- What we currently have:
- Written in 2010 at the time of PHP 5.3 standards (12 years ago, which matched that standards at the time of it was written)
- Unmanaged and difficult to update and manage as it's hard-coded, covering a small subset of rules
- How does a linting tool help us?
- Less time spent on syntax, more on implementing and pushing solutions (towards PHP 8 compliance).
- Easily to change, update, and adapt the rules within the project as we need and in our speed.
- Eventually, CI/CD can warn us of issues of new code we push the code before merging, i.e. on GitLab
- With a standardised syntax, the core team can more easily work with community patches and get them in faster.
- What is a linter and formatter?
- A linter is a static analysis tool that can show us area of bad coding practice.
- A static analysis tool to find weak spots in the code, highlighting issues
- where rules are configurable
- We can change the strictness, warnings vs. errors.
- Formatters provide syntax checking and updates code
- How does it help other teams at Catalyst?
- The frontend team use Prettier formatter – a set of rules to follow
- Fix once, get the benefits going forwards with minimal changes
- The frontend team agreed to use Airbnb style guide to help use best practices, but there are many others to choose from.
- Look into prettier for formatter, ESLint for linter. When starting on ESLint, it asks you for the rules you like.
- We could have it as a tool that formats the code, so we can diff before pushing change
- What we currently have:
- Doris Tam (Wellington, NZ) - Mahara developer, Catalyst IT
- Gold (Wellington, NZ) - Mahara developer, Catalyst IT
- Kristina Hoeppner (Wellington, NZ), Mahara project lead, Catalyst IT
- Adam Bark (Texas, US), Co-founder, LearnOpenTech
- Robert Lyon
- `make minaccept` updates : postponed for Bob being present
- `make security` updates: still needs attention
- PHP 8.2 compatibility
- Mahara Core has no Errors or Warnings
- Third party libraries in the Mahara codebase still have some issues, but not a lot. Updates may resolve these.
- Project wide code linter
- A JS linter will also be needed but will discuss in the future
- Extra discovery has been done and captured as this proposal as well.
- Other business
- Next Developer Meeting
- Who: Bob (Chair), Kristina (Note taker)
- When: Wednesday, 22 March 2023 at 23:00 UTC