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

Replace Ace Editor with CodeMirror #4933

Merged
merged 4 commits into from
Sep 12, 2023

Remove italic from variable names

2e4cd2b
Select commit
Loading
Failed to load commit list.
Merged

Replace Ace Editor with CodeMirror #4933

Remove italic from variable names
2e4cd2b
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Sep 12, 2023 in 1s

59.57% of diff hit (target 77.80%)

View this Pull Request on Codecov

59.57% of diff hit (target 77.80%)

Annotations

Check warning on line 6 in app/assets/javascripts/coding_scratchpad.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/coding_scratchpad.ts#L6 

Added line #L6 was not covered by tests

Check warning on line 50 in app/assets/javascripts/coding_scratchpad.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/coding_scratchpad.ts#L50 

Added line #L50 was not covered by tests

Check warning on line 3 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L1-L3 

Added lines #L1 - L3 were not covered by tests

Check warning on line 12 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L11-L12 

Added lines #L11 - L12 were not covered by tests

Check warning on line 22 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L22 

Added line #L22 was not covered by tests

Check warning on line 141 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L140-L141 

Added lines #L140 - L141 were not covered by tests

Check warning on line 149 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L148-L149 

Added lines #L148 - L149 were not covered by tests

Check warning on line 157 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L156-L157 

Added lines #L156 - L157 were not covered by tests

Check warning on line 170 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L170 

Added line #L170 was not covered by tests

Check warning on line 1 in app/assets/javascripts/exercise.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L1 

Added line #L1 was not covered by tests

Check warning on line 159 in app/assets/javascripts/exercise.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L159 

Added line #L159 was not covered by tests

Check warning on line 173 in app/assets/javascripts/exercise.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L173 

Added line #L173 was not covered by tests

Check warning on line 179 in app/assets/javascripts/exercise.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L179 

Added line #L179 was not covered by tests
View more details on Codecov