Skip to content

Create swift.yml - #13

Open
Roberto1401 wants to merge 2 commits into
coolstar:masterfrom
Roberto1401:Roberto1401-patch-1
Open

Create swift.yml#13
Roberto1401 wants to merge 2 commits into
coolstar:masterfrom
Roberto1401:Roberto1401-patch-1

Conversation

@Roberto1401

Copy link
Copy Markdown

No description provided.

@Roberto1401 Roberto1401 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Good

@Roberto1401 Roberto1401 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Senks

Comment on lines +1 to +19
name: Swift

on:
push:
branches: [ master ]
pull_request:
branches: [ master ]

jobs:
build:

runs-on: macos-latest

steps:
- uses: actions/checkout@v2
- name: Build
run: swift build -v
- name: Run tests
run: swift test -v

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
name: Swift
on:
push:
branches: [ master ]
pull_request:
branches: [ master ]
jobs:
build:
runs-on: macos-latest
steps:
- uses: actions/checkout@v2
- name: Build
run: swift build -v
- name: Run tests
run: swift test -v
name: Swift
on:
push:
branches: [ master ]
pull_request:
branches: [ master ]
jobs:
build:
runs-on: macos-latest
steps:
- uses: actions/checkout@v2
- name: Build
run: swift build -v
- name: Run tests
run: swift test -v

@suprstarrd

Copy link
Copy Markdown

This will never work.

With the Actions set here, there is no root manifest set. ...Which this project doesn't have.

If I was to actually try to make a root manifest here, it would still fail.

I'll try to see what I can do, but I am almost positive this will not work.

@suprstarrd suprstarrd left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This doesn't actually build Chimera13. At all. And it's been untouched for ages now. Please fix it.

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.

6 participants