Switch public class properties for constants and private/protected properties
@GaryJones is already working on this.
Since Aug 16, 2024.
Assessment
This issue has not been assessed yet.
Description
Review the public properties and see which can be changed to constants, and which can be made non-public.
While this would be a breaking change, we're still on major version 0.x.y, so breaking changes are allowed.
- Dominant language
- PHP
- Stars
- 86
- Forks
- 53
- PR merge metrics
- No merged PRs in 30d
Getting set up
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Automattic/zoninator
-
type: maintenance
Difficulty 4/5 3-5 days Newbie friendliness 35/100
Automattic/zoninator#122 ·
-
Fix legacy use of get_terms()May be free again @GaryJones claimed this 772 days ago, and no pull request is open. Opentype: maintenance
Automattic/zoninator#111 · 1 assignee ·
-
Remove use of wpcom_vip_get_term_by()May be free again @GaryJones claimed this 772 days ago, and no pull request is open. Opentype: maintenance
Automattic/zoninator#110 · 1 assignee ·
-
Implement cachingOpentype: enhancement
Difficulty 4/5 3-5 days Newbie friendliness 30/100
Automattic/zoninator#105 ·
-
Finalize classesOpentype: maintenance
Difficulty 4/5 3-5 days Newbie friendliness 30/100
Automattic/zoninator#104 ·
All issues in Automattic/zoninator
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Automattic/static-site-importer#1879 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
521xueweihan/HelloGitHub#3790 ·
-
[Bug] Feed date, title and author too long to fit inside article box on smaller screens, mobileOpenBug (unconfirmed) Good first issue 1️⃣ help wanted UI :art:
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
FreshRSS/FreshRSS#9360 · 2 comments ·
Maintainers usually reply within 1 day