Skip to content

Conversation

@mbeisel
Copy link
Contributor

@mbeisel mbeisel commented May 14, 2020

Merge for a stable version of the latexrendering component.

@mbeisel mbeisel requested a review from manuwei May 14, 2020 16:42
@ghost
Copy link

ghost commented May 14, 2020

DeepCode's analysis on #c84ee6 found:

1 minor issue.

Top 3 issues

Description of the issue ❌ Example fixes
Printing the stack trace of java.lang.Exception. Production code should not use printStackTrace. Occurrences: Example fixes 🔧


👉 View analysis in DeepCode’s Dashboard
Edit DeepCode’s Configurations here

☺️ If you want to provide feedback on our bot, here is how to contact us.

Copy link
Contributor

@manuwei manuwei left a comment

Choose a reason for hiding this comment

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

Hey @mbeisel, in general the PR looks good.
Can you solve the merge conflicts and have a look at the DeepCode's suggestions?


// @Transactional (readOnly = false)
// String renderLatex(String content, int begin, int end, AlgorithmType type);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this commented code still needed for something?

@manuwei
Copy link
Contributor

manuwei commented May 27, 2020

@mbeisel Can you delete the data.sql files? we've decided to save data in a separate repo (https://github.com/PatternPedia/pattern-atlas-content or https://github.com/PatternPedia/internal-pattern-atlas-content)

And our ci tests still fail....

@manuwei manuwei closed this May 27, 2020
@manuwei manuwei reopened this May 27, 2020
@ghost
Copy link

ghost commented Jun 23, 2020

Congratulations 🎉. DeepCode analyzed your code in 3.532 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@manuwei
Copy link
Contributor

manuwei commented Jul 6, 2020

@mbeisel can you fix the critical issue the deepcode-ci-bot found?

@manuwei manuwei merged commit 8605b7a into master Oct 20, 2020
@lharzenetter lharzenetter deleted the masterthesis/patternauthoringAPI branch June 23, 2021 14:55
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.

3 participants