Skip to content

Reserved core tests in production #116

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Oct 28, 2024
Merged

Reserved core tests in production #116

merged 6 commits into from
Oct 28, 2024

Conversation

Gioee
Copy link
Member

@Gioee Gioee commented Oct 23, 2024

Description of change

Core tests are now in a unique core.test.ts file, with new reserved command tests.

Pull-Request Checklist

  • Code is up-to-date with the main branch
  • npm run lint passes with this change
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000 N/A
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change N/A
  • The new commits follow conventions outlined in the conventional commit spec

@Gioee Gioee added the enhancement New feature or request label Oct 23, 2024
@Gioee Gioee requested a review from danielebriggi October 23, 2024 13:28
@Gioee Gioee self-assigned this Oct 23, 2024
@Gioee Gioee requested review from andinux and removed request for danielebriggi October 23, 2024 13:29
@Gioee Gioee requested review from TizianoT and removed request for andinux October 28, 2024 14:56
@Gioee Gioee merged commit 0b48da4 into main Oct 28, 2024
2 checks passed
@Gioee Gioee deleted the fix-rn-buffer-and-core-tests branch October 28, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TizianoT TizianoT TizianoT approved these changes

Assignees

@Gioee Gioee

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Gioee @TizianoT