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

Warnings generating a new project #334

Closed
pepicrft opened this issue Apr 28, 2019 · 1 comment · Fixed by #339
Closed

Warnings generating a new project #334

pepicrft opened this issue Apr 28, 2019 · 1 comment · Fixed by #339
Labels
type:bug Something isn't working

Comments

@pepicrft
Copy link
Contributor

Context 🕵️‍♀️

I just updated to the latest version and ran the following commands:

  • tuist init
  • tuist focus

Expected
Get the project generated without any warnings on the console

Got
image

@pepicrft pepicrft added type:bug Something isn't working domain:tuist labels Apr 28, 2019
@kwridan
Copy link
Collaborator

kwridan commented May 1, 2019

Ah this due to the changes I introduced for #318 where a warning is displayed if the glob patterns specified doesn't yield any files. It was perhaps too strict, we can remove it.

https://github.com/tuist/tuist/blob/master/Sources/TuistKit/Generator/GeneratorModelLoader.swift#L112

kwridan added a commit to bloomberg/tuist that referenced this issue May 5, 2019
Resolves tuist#334

- The path to resources is now commented out and left as an example to aid users get started defining the manifest
- The warnings were caused due to the `Resources`  directory not existing

Test Plan:

- Create a new empty directory
- Run `tuist init`
- Run `tuist focus`
- Verify no warnings are displayed
kwridan added a commit that referenced this issue May 5, 2019
Resolves #334

- The path to resources is now commented out and left as an example to aid users get started defining the manifest
- The warnings were caused due to the `Resources`  directory not existing

Test Plan:

- Create a new empty directory
- Run `tuist init`
- Run `tuist focus`
- Verify no warnings are displayed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants