Skip to content

Commit 144d5af

Browse files
authored
Merge pull request #14 from dannyhp1/develop
Remove limit on logs.
2 parents 5a4c00b + e2c8be4 commit 144d5af

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

src/App/App.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,9 @@ class App extends Component {
152152
})
153153
}
154154

155+
/**
156+
* Downloads the currently displayed source code to the user's desktop.
157+
*/
155158
downloadCode = () => {
156159
// TODO: Code to download source code.
157160
}
@@ -164,14 +167,9 @@ class App extends Component {
164167
// Before processing the new result, we must pop the message 'Running your code...' off and re-enable the button.
165168
this.setFinishedStatus()
166169

167-
// Will only keep the latest 10 results in the logs.
170+
// TODO: Keep only a specific amount of logs.
168171
let currentResults = this.state.results
169172

170-
// Most recent results are at the top, remove the element at the end if we exceed 10 logs.
171-
if (currentResults.length >= 10) {
172-
currentResults.pop()
173-
}
174-
175173
if (result['build'] === true) {
176174
currentResults.unshift('[' + languages[this.state.language] + '] Build successfully completed!\nStandard output:\n' + result['message'])
177175
} else {

0 commit comments

Comments
 (0)