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.

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

fix: added more gremlin characters #191

Merged
merged 8 commits into from
Dec 30, 2022
Merged

fix: added more gremlin characters #191

merged 8 commits into from
Dec 30, 2022

Conversation

sabhas
Copy link
Member

@sabhas sabhas commented Dec 27, 2022 •
edited
Loading

Issue

closes #190

Checks

  • Code is formatted correctly (npm run lint:fix).
  • Any new functionality has been unit tested.
  • All unit tests are passing (npm test).
  • All CI checks are green.
  • sasjslint-schema.json is updated with any new / changed functionality
  • JSDoc comments have been added or updated.
  • Reviewer is assigned.

@sasjsbot sasjsbot requested a review from saadjutt01 December 27, 2022 18:41
@allanbowe allanbowe merged commit f59fd4c into main Dec 30, 2022
@allanbowe allanbowe deleted the issue-190 branch December 30, 2022 10:49
@sasjsbot
Copy link

🎉 This PR is included in version 2.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@allanbowe allanbowe allanbowe approved these changes

@medjedovicm medjedovicm medjedovicm approved these changes

@saadjutt01 saadjutt01 Awaiting requested review from saadjutt01

Assignees

@sabhas sabhas

Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

More Gremlins
4 participants
@sabhas @sasjsbot @allanbowe @medjedovicm