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

Removed TODO comments (tickets created) #48

Merged
merged 2 commits into from
Jan 22, 2025
Merged

Removed TODO comments (tickets created) #48

merged 2 commits into from
Jan 22, 2025

Conversation

shayfaber
Copy link
Contributor

Removed TODO comments (tickets created), remove empty tests, and add Gherkin description to a test.

@shayfaber shayfaber requested a review from dheesen January 20, 2025 11:13
@shayfaber
Copy link
Contributor Author

@dheesen I had to cancel this check because we were working in the admin account, which caused an error in this workflow.

dheesen
dheesen previously approved these changes Jan 22, 2025
@dheesen dheesen merged commit 58fb7e8 into main Jan 22, 2025
1 check failed
@dheesen dheesen deleted the todo-comments branch January 22, 2025 14:07
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.

2 participants