Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

EPIC: Code quality #101

Closed
blahosyl opened this issue Aug 7, 2024 · 0 comments
Closed

EPIC: Code quality #101

blahosyl opened this issue Aug 7, 2024 · 0 comments
Assignees
@blahosyl
Labels
e: code quality Epic: Code quality epic ongoing Tasks performed throughout development p: must have Priority: Part of the MVP p: should have Priority: High value but not crucial refactoring Code refactoring without changes in functionaity
Milestone
Sprint 4: Testing...

Comments

@blahosyl
Copy link
Owner

blahosyl commented Aug 7, 2024 •
edited
Loading

As a Developer,

I want to write code that is easy to navigate and DRY,

so that it can be reused and audited better

Acceptance Criteria

  1. Consistent indentation promotes legibility
  2. Conforms to DRY principle
  3. Sufficient comments & docstrings
  4. Validated with relevant tools

Issues in this Epic:

  1. HTML code quality #153
  2. USER STORY: CSS code quality #154
  3. USER STORY: USER STORY: JavaScript & JSX code quality #155
  4. USER STORY: Python code quality task-manager-api#22
@blahosyl blahosyl self-assigned this Aug 7, 2024
@blahosyl blahosyl moved this to Blocked/Ongoig in Task Manager Aug 7, 2024
@blahosyl blahosyl added this to the Sprint 2: Profiles & UX milestone Aug 7, 2024
@blahosyl blahosyl added ongoing Tasks performed throughout development p: should have Priority: High value but not crucial refactoring Code refactoring without changes in functionaity labels Aug 7, 2024
blahosyl added a commit that referenced this issue Aug 7, 2024
@blahosyl
move Rounded CSS to App styles #101
798c9bf
blahosyl added a commit that referenced this issue Aug 7, 2024
@blahosyl
change placeholder capitalization in SignIn/Up form #101
dc5a163 
this unifies capitalization with the ChangePasswordForm
blahosyl added a commit that referenced this issue Aug 8, 2024
@blahosyl
replace class with className #101
89a8b80
blahosyl added a commit that referenced this issue Aug 8, 2024
@blahosyl
remove typo #101
d767e2f
blahosyl added a commit that referenced this issue Aug 8, 2024
@blahosyl
remove unused imports #101 #16
916be44 
no change in functionality
blahosyl added a commit that referenced this issue Aug 8, 2024
@blahosyl
formatting & code comments #101 #16
102894d
blahosyl added a commit that referenced this issue Aug 9, 2024
@blahosyl
move color var definitions to App.module.css #101
5ab07ad
blahosyl added a commit that referenced this issue Aug 9, 2024
@blahosyl
update code comments in ProfileDataContext #101
d02377c
blahosyl added a commit that referenced this issue Aug 10, 2024
@blahosyl
fix code indentation #101
5700143 
no change in functionality
blahosyl added a commit that referenced this issue Aug 11, 2024
@blahosyl
format code & remove superfluous commented lines #101 #66
c4329aa
blahosyl added a commit that referenced this issue Aug 11, 2024
@blahosyl
update comments #66 #101
053aede
blahosyl added a commit that referenced this issue Aug 12, 2024
@blahosyl
add code comments to ProfileList #101
4ffe4ec
blahosyl added a commit that referenced this issue Aug 12, 2024
@blahosyl
remove unused import #101
c7ba4e4
blahosyl added a commit that referenced this issue Aug 12, 2024
@blahosyl
rename list prop to taskList, pass it down to Task #101
6d3a3a9 
no change in functionality
blahosyl added a commit that referenced this issue Aug 13, 2024
@blahosyl
add code comment #101
22559ad
blahosyl added a commit that referenced this issue Aug 14, 2024
@blahosyl
add code comment #101
f7f62b0
blahosyl added a commit that referenced this issue Aug 14, 2024
@blahosyl
update code comment #101
141760f
blahosyl added a commit that referenced this issue Aug 16, 2024
@blahosyl
remove unused code block #101
681695f
@blahosyl blahosyl added the p: must have Priority: Part of the MVP label Aug 16, 2024
@blahosyl blahosyl added the e: code quality Epic: Code quality label Aug 16, 2024
blahosyl added a commit that referenced this issue Aug 17, 2024
@blahosyl
correct spelling #101
392f0ab
blahosyl added a commit that referenced this issue Aug 17, 2024
@blahosyl
delete unused code block #101
429c30c
blahosyl added a commit that referenced this issue Aug 17, 2024
@blahosyl
remove commented-out code #101
35c0e40
blahosyl added a commit that referenced this issue Aug 19, 2024
@blahosyl
correct typo #101
f2b50f5
@blahosyl blahosyl added the epic label Aug 19, 2024
@blahosyl blahosyl changed the title USER STORY: Code quality EPIC: Code quality Aug 19, 2024
blahosyl added a commit that referenced this issue Aug 19, 2024
@blahosyl
customize Markdownlint warnings #101
8195ec8
blahosyl added a commit that referenced this issue Aug 19, 2024
@blahosyl
correct Bootstrap class declaration #101
f7004f7
@github-project-automation github-project-automation bot moved this from Blocked/Ongoig to Done/Won't do in Task Manager Aug 20, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@blahosyl blahosyl

Labels
e: code quality Epic: Code quality epic ongoing Tasks performed throughout development p: must have Priority: Part of the MVP p: should have Priority: High value but not crucial refactoring Code refactoring without changes in functionaity
Projects
Task Manager
Status: Done/Won't do
Milestone
Sprint 4: Testing & documentation
Development

No branches or pull requests
1 participant
@blahosyl