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

upgrade ace.js==1.15.0 #2045

Merged
merged 2 commits into from
Feb 9, 2023
Merged

upgrade ace.js==1.15.0 #2045

merged 2 commits into from
Feb 9, 2023

Conversation

Yebolin
Copy link
Contributor

@Yebolin Yebolin commented Feb 8, 2023

upgrade ace.js==1.15.0
#375

@codecov
Copy link

codecov bot commented Feb 8, 2023

Codecov Report

Base: 75.03% // Head: 75.03% // No change to project coverage 👍

Coverage data is based on head (5d3db65) compared to base (4246273).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2045   +/-   ##
=======================================
  Coverage   75.03%   75.03%           
=======================================
  Files         102      102           
  Lines       14809    14809           
=======================================
  Hits        11112    11112           
  Misses       3697     3697           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@LeoQuote
Copy link
Collaborator

LeoQuote commented Feb 8, 2023

rebase 下, 解决下 lint 问题

Add files via upload

Add files via upload
@Yebolin
Copy link
Contributor Author

Yebolin commented Feb 8, 2023

rebase 下, 解决下 lint 问题

我 rebase 了还是一样,lint 报错的那几个文件,我根本没更新到。

@LeoQuote
Copy link
Collaborator

LeoQuote commented Feb 8, 2023

在这里 #2046 修了, 这个pr 可以安全 merge

@LeoQuote
Copy link
Collaborator

LeoQuote commented Feb 8, 2023

前端部分已经测试过了吗? CI 只能跑后端的

@Yebolin
Copy link
Contributor Author

Yebolin commented Feb 9, 2023

前端部分已经测试过了吗? CI 只能跑后端的

我没有很严格的测试。文件只更新了ace编辑器。
我测试了手机跟PC。手机浏览器确实可以粘贴了。
然后pc我做日常操作,控制台没有js相关的报错

Copy link
Collaborator

@LeoQuote LeoQuote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

前端大概测试下就行,功能完备就行了

@LeoQuote LeoQuote merged commit 0eee241 into hhyo:master Feb 9, 2023
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