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

Issue 1019: Improve Annalyn's infiltration by making the tests declarative #1242

Merged
merged 5 commits into from
Aug 1, 2021
Merged

Issue 1019: Improve Annalyn's infiltration by making the tests declarative #1242

merged 5 commits into from
Aug 1, 2021

Conversation

AbhilashJN
Copy link
Contributor

No description provided.

@SleeplessByte
Copy link
Member

Could you please add a space between the =s and variable names, and values, so it's consistent with the rest?

- const foo=true
+ const foo = true

@AbhilashJN
Copy link
Contributor Author

I have updated the PR

@SleeplessByte
Copy link
Member

/format

@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2021

The "Format code" action has started running.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2021

The "Format code" action has finished running.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2021

For security reasons, /format does not trigger CI builds when the PR has been submitted from a fork. If checks were not passing due to code format, trigger a build to make the required checks pass, through one of the following ways:

  • Push an empty commit to this branch: git commit --allow-empty -m "Trigger builds".
  • Close and reopen the PR.
  • Push a regular commit to this branch.

@SleeplessByte SleeplessByte reopened this Aug 1, 2021
@SleeplessByte SleeplessByte merged commit b850bb1 into exercism:main Aug 1, 2021
@SleeplessByte
Copy link
Member

Thank you @AbhilashJN !!

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