Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Revert "Revert "modified lint errors with comments out and added the last code to make the code full"" #72

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

raissg
Copy link

@raissg raissg commented Sep 26, 2024

Reverts #46

Important Note: This pull request reverts a previously reverted change, restoring the feature to its original state. The purpose is to ensure that the correct reviewers are assigned to each task we implemented. Due to a repository configuration issue, automatic merging occurred before the assigned reviewers had officially approved the code. Although the reviewers completed their reviews on time, GitHub merged the code prematurely.

This "undo revert" pull request restores the changes from the original pull request #42. Please refer to that pull request to view the commit messages, as they are not displayed here. The reviewers from that pull request #42 can be seen here.

@raissg raissg added duplicate This issue or pull request already exists enhancement New feature or request labels Sep 26, 2024
@raissg raissg added this to the Sprint 1 milestone Sep 26, 2024
@raissg raissg requested review from Seckhen and NickDevi September 26, 2024 17:39
@raissg raissg self-assigned this Sep 26, 2024
Copy link

@Seckhen Seckhen left a comment

Choose a reason for hiding this comment

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

All good, ready to merge!

Copy link

@NickDevi NickDevi left a comment

Choose a reason for hiding this comment

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

All good, ready for merging.

@raissg raissg merged commit 02cb1d9 into f24 Sep 26, 2024
1 check passed
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11057416415

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 82.693%

Totals Coverage Status
Change from base Build 11057373409: 0.005%
Covered Lines: 22343
Relevant Lines: 25596

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants