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

The branch updatedPinReply has been modified #44

Merged
merged 65 commits into from
Feb 22, 2025
Merged

Conversation

Jdwashin9
Copy link
Contributor

Adds the onPinClicked tests and assures the submodule references provide the correct URL. This code leverages the testing framework known as "Jest", and these commits were written with the assistance of CHATGPT4/Github Copilot.

Jdwashin9 added 30 commits February 22, 2025 02:08
Removed all errors from test/uploads.js
Test/topics erros fixed

test/template-helpers errors fixed
Fixed test/posts

test/plugins errors resolved

Fixed password and pagination in test

fixed posts/uploads in test

notifications fixed in test

middleware fixed in test

Fixed test/meta
fixed all test files from categories to flags

fixed api through build in test and reolved warning

Completely fixed test/database

updated .gitmodules

submodule in NodeBB altered.

NodeBB .gitmodule error and polished npm lint again

created .gitmodules submodule for repo
Updated categories 1543 to boost coverage

Categories change 1550

Categories update 1600
Jdwashin9 added 25 commits February 22, 2025 02:21
2341 wrapper

ensured that keys handles an expected value error for 65
statements/functions coverage increase by 0.05

got workflow working again by converging the two jobs sections

Resolved lint error 0509

yaml changed to add memory commit to redis

0614 updates
Adding import statements for Jest global

properly named addImportStatement file

commiting changes to establish ImportStatement in origin/PinReply

fixed json paths in new dev_container
@Jdwashin9 Jdwashin9 closed this Feb 22, 2025
@Jdwashin9 Jdwashin9 reopened this Feb 22, 2025
@Jdwashin9
Copy link
Contributor Author

I NEED to change the workflow file. That's how the memory overcommit issue can be resolved.

@Jdwashin9 Jdwashin9 merged commit 28e2060 into main Feb 22, 2025
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.

1 participant