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

Replace Ace Editor with CodeMirror 6 #4970

Merged
merged 13 commits into from
Oct 30, 2023

Make init more async

e1b00b0
Select commit
Loading
Failed to load commit list.
Merged

Replace Ace Editor with CodeMirror 6 #4970

Make init more async
e1b00b0
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Oct 27, 2023 in 1s

60.41% of diff hit (target 77.69%)

View this Pull Request on Codecov

60.41% of diff hit (target 77.69%)

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 14 in app/assets/javascripts/editor.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L13-L14 

Added lines #L13 - L14 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L24 

Added line #L24 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L150-L151 

Added lines #L150 - L151 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L158-L159 

Added lines #L158 - L159 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L166-L167 

Added lines #L166 - L167 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/editor.ts#L180 

Added line #L180 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 160 in app/assets/javascripts/exercise.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L160 

Added line #L160 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L174 

Added line #L174 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/exercise.ts#L180 

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