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 build issue #879

Merged
merged 2 commits into from
Jul 28, 2023
Merged

Fix build issue #879

merged 2 commits into from
Jul 28, 2023

Conversation

dsl101
Copy link
Contributor

@dsl101 dsl101 commented Jul 28, 2023

PR Creator Checklist

Ensure you've checked the following before submitting your PR:

  • You've discussed making your changes with a member of the dev team per contributing rules in the README
  • Your changes are free of any lint errors
  • Your changes are free of any typescript errors
  • You've tested your changes

Summary

Please provide a summary of what your PR does

Fixes obscure build issue where the blank line causes one of the scripts to fail with command not found: .

Test Plan

Please document the steps required to test your PR

Build on other developer's environments to ensure all good.

@gkasdorf gkasdorf merged commit 5600476 into Memmy-App:main Jul 28, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dsl101 @gkasdorf