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

Add Hello World #36

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

Add Hello World #36

wants to merge 45 commits into from

Conversation

JakeIV
Copy link
Collaborator

@JakeIV JakeIV commented Oct 19, 2022

Checklist ✅

  • Have you checked to see whether there are any conflicting / similar PRs already open?
  • Does your PR add any new tests
  • Does your PR pass all existing & new tests
  • Does your PR pass the linter?

What does this PR do? 🤔

Screenshots / Videos 📸

web/src/App.tsx Outdated Show resolved Hide resolved
web/src/App.tsx Outdated
@@ -3,6 +3,12 @@ import './App.css';
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';

function helloWorld(){
console.log("Hello World")

Choose a reason for hiding this comment

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

🚫 [eslint] <quotes> reported by reviewdog 🐶
Strings must use singlequote.

Suggested change
console.log("Hello World")
console.log('Hello World')

web/src/App.tsx Outdated
@@ -3,6 +3,12 @@ import './App.css';
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';

function helloWorld(){
console.log("Hello World")

Choose a reason for hiding this comment

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

🚫 [eslint] <semi> reported by reviewdog 🐶
Missing semicolon.

Suggested change
console.log("Hello World")
console.log("Hello World");

web/src/App.tsx Outdated
console.log("Hello World")
}

helloWorld()

Choose a reason for hiding this comment

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

🚫 [eslint] <semi> reported by reviewdog 🐶
Missing semicolon.

Suggested change
helloWorld()
helloWorld();

web/src/App.tsx Outdated Show resolved Hide resolved
JakeIV and others added 3 commits October 19, 2022 16:55
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
web/src/App.tsx Outdated
Comment on lines 6 to 7
class SnakeEngine{

Choose a reason for hiding this comment

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

🚫 [eslint] <padded-blocks> reported by reviewdog 🐶
Block must not be padded by blank lines.

Suggested change
class SnakeEngine{
class SnakeEngine{

web/src/App.tsx Outdated Show resolved Hide resolved
web/src/App.tsx Outdated Show resolved Hide resolved
web/src/App.tsx Outdated Show resolved Hide resolved
web/src/App.tsx Outdated
class SnakeEngine{

static helloWorld(){
console.log("Hello World")

Choose a reason for hiding this comment

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

🚫 [eslint] <semi> reported by reviewdog 🐶
Missing semicolon.

Suggested change
console.log("Hello World")
console.log("Hello World");

web/src/App.tsx Outdated Show resolved Hide resolved
Jonas-Oppong and others added 4 commits October 19, 2022 17:04
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
web/src/App.tsx Outdated
Comment on lines 6 to 7
class SnakeEngine{

Choose a reason for hiding this comment

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

🚫 [eslint] <padded-blocks> reported by reviewdog 🐶
Block must not be padded by blank lines.

Suggested change
class SnakeEngine{
class SnakeEngine

web/src/App.tsx Outdated
@@ -3,6 +3,15 @@ import './App.css';
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';

class SnakeEngine{

Choose a reason for hiding this comment

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

🚫 [eslint] <space-before-blocks> reported by reviewdog 🐶
Missing space before opening brace.

Suggested change
class SnakeEngine{
class SnakeEngine {

web/src/App.tsx Outdated
@@ -3,6 +3,15 @@ import './App.css';
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';

class SnakeEngine{

static helloWorld(){

Choose a reason for hiding this comment

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

🚫 [eslint] <space-before-blocks> reported by reviewdog 🐶
Missing space before opening brace.

Suggested change
static helloWorld(){
static helloWorld() {

web/src/App.tsx Outdated Show resolved Hide resolved
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
web/src/App.tsx Outdated
Comment on lines 6 to 7
class SnakeEngine{

Choose a reason for hiding this comment

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

🚫 [eslint] <padded-blocks> reported by reviewdog 🐶
Block must not be padded by blank lines.

Suggested change
class SnakeEngine{
class SnakeEngine

web/src/App.tsx Outdated
@@ -3,6 +3,15 @@ import './App.css';
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';

class SnakeEngine{

Choose a reason for hiding this comment

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

🚫 [eslint] <space-before-blocks> reported by reviewdog 🐶
Missing space before opening brace.

Suggested change
class SnakeEngine{
class SnakeEngine {

web/src/App.tsx Outdated
class SnakeEngine{

static helloWorld() {
console.log('Hello World')

Choose a reason for hiding this comment

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

🚫 [eslint] <semi> reported by reviewdog 🐶
Missing semicolon.

Suggested change
console.log('Hello World')
console.log('Hello World');;

web/src/App.tsx Outdated
Comment on lines 6 to 7
class SnakeEngine {

Choose a reason for hiding this comment

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

🚫 [eslint] <padded-blocks> reported by reviewdog 🐶
Block must not be padded by blank lines.

Suggested change
class SnakeEngine {
class SnakeEngine {

web/src/App.tsx Outdated
class SnakeEngine {

static helloWorld() {
console.log('Hello World')

Choose a reason for hiding this comment

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

🚫 [eslint] <semi> reported by reviewdog 🐶
Missing semicolon.

Suggested change
console.log('Hello World')
console.log('Hello World');;

web/src/App.tsx Outdated
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';
import {SnakeEngine} from './SnakeEngine/SnakeEngine';

Choose a reason for hiding this comment

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

🚫 [eslint] <object-curly-spacing> reported by reviewdog 🐶
A space is required after '{'.

Suggested change
import {SnakeEngine} from './SnakeEngine/SnakeEngine';
import { SnakeEngine} from './SnakeEngine/SnakeEngine';

web/src/App.tsx Outdated Show resolved Hide resolved
JakeIV and others added 4 commits October 19, 2022 17:27
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
web/src/App.tsx Outdated
import reactLogo from './assets/react.svg';
import IncrementButton from './IncrementButton';
import {SnakeEngine } from './SnakeEngine/SnakeEngine';

Choose a reason for hiding this comment

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

🚫 [eslint] <object-curly-spacing> reported by reviewdog 🐶
A space is required after '{'.

Suggested change
import {SnakeEngine } from './SnakeEngine/SnakeEngine';
import { SnakeEngine } from './SnakeEngine/SnakeEngine';

@JavaRip
Copy link
Contributor

JavaRip commented Oct 23, 2022

Great start, please merge main into this branch to resolve conflicts then will approve

Thanks!

@JavaRip JavaRip self-requested a review October 23, 2022 13:33
Copy link
Contributor

@JavaRip JavaRip left a comment

Choose a reason for hiding this comment

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

Great work! Merge main into branch to resolve conflicts then can approve

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.

[New Feature]: Controller / direction input 🎮
5 participants