Skip to content

修灯#326

Open
chenxi1916 wants to merge 2 commits intomasterfrom
chenxi
Open

修灯#326
chenxi1916 wants to merge 2 commits intomasterfrom
chenxi

Conversation

@chenxi1916
Copy link
Copy Markdown
Contributor

  • 这个PR解答了当前仓库中的题目(机器人会自动判题并合并当前PR)
  • 这个PR修复了当前仓库中的一些代码缺陷(机器人不会判题,而是由管理员来处理当前PR)

@hcsp-bot
Copy link
Copy Markdown
Contributor

🎉 感谢提交Pull Request!请稍等片刻,我们已经将其提交到CI进行检查,一旦有结果会立即通知您!

@hcsp-bot
Copy link
Copy Markdown
Contributor

你的提交 c8168d3 ,似乎失败了:Could not find a usable config.yml, you may have revoked the CircleCI OAuth app.

😅 请不要气馁,仔细分析原因,再接再厉!

// 当灯亮时返回true,灭和未知状态返回false
public boolean isOn() {
return on;
if (on == null || on == false){
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

不必要的条件逻辑。
表达式可被简化。
'{' 前应有空格。

@hcsp-bot
Copy link
Copy Markdown
Contributor

你的提交 9073434 ,似乎失败了:Could not find a usable config.yml, you may have revoked the CircleCI OAuth app.

😅 请不要气馁,仔细分析原因,再接再厉!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants