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

3️⃣ Merge tuist focus and tuist generate command #3912

Merged
merged 31 commits into from
Jan 19, 2022

Conversation

danieleformichelli
Copy link
Collaborator

@danieleformichelli danieleformichelli commented Dec 28, 2021

No description provided.

@danieleformichelli danieleformichelli added this to the 3.0 milestone Dec 28, 2021
Copy link
Collaborator

@luispadron luispadron left a comment

Choose a reason for hiding this comment

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

This is a great direction!

@adellibovi
Copy link
Collaborator

As commented on Slack, maybe we can soft-deprecate the command and redirect tuist focus to tuist generate? In that case we don't really need to wait for a 3.0. What do you think?

@danieleformichelli danieleformichelli changed the base branch from main to release/3.0 January 3, 2022 13:52
@danieleformichelli danieleformichelli marked this pull request as ready for review January 3, 2022 13:52
Co-authored-by: ezraberch <49635435+ezraberch@users.noreply.github.com>
# Conflicts:
#	CHANGELOG.md
#	Sources/TuistKit/Commands/TuistCommand.swift
#	projects/fourier/lib/fourier.rb
#	projects/fourier/lib/fourier/services/focus.rb
#	projects/fourier/test/fourier/services/focus_test.rb
#	projects/fourier/test/fourier/services/generate/tuist_test.rb
#	projects/tuist/features/step_definitions/shared/tuist.rb
Sources/TuistKit/Commands/GenerateCommand.swift Outdated Show resolved Hide resolved
Sources/TuistKit/Commands/GenerateCommand.swift Outdated Show resolved Hide resolved
Sources/TuistKit/Commands/GenerateCommand.swift Outdated Show resolved Hide resolved
projects/docs/docs/commands/generate.md Outdated Show resolved Hide resolved
projects/docs/docs/commands/generate.md Outdated Show resolved Hide resolved
projects/docs/docs/commands/generate.md Outdated Show resolved Hide resolved
projects/docs/docs/commands/generate.md Show resolved Hide resolved
@fortmarek
Copy link
Member

As commented on Slack, maybe we can soft-deprecate the command and redirect tuist focus to tuist generate? In that case we don't really need to wait for a 3.0. What do you think?

I'd rather make the change in one go as it might be wasted effort to support both tuist generate XYZ and tuist focus XYZ.

Co-authored-by: Marek Fořt <marekfort@me.com>
@danieleformichelli danieleformichelli changed the title 3️⃣ [WIP] Merge tuist focus and tuist generate command 3️⃣ Merge tuist focus and tuist generate command Jan 19, 2022
Copy link
Member

@fortmarek fortmarek left a comment

Choose a reason for hiding this comment

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

I'm good with this approach, thanks for working on this @danyf90 👏

@danieleformichelli danieleformichelli merged commit b29e347 into release/3.0 Jan 19, 2022
@danieleformichelli danieleformichelli deleted the feature/merge-generate-focus branch January 19, 2022 21:17
danieleformichelli added a commit that referenced this pull request Jan 28, 2022
* 3️⃣ Remove deprecated deprecated initializers for `FileLists`, `Headers`, and `HTTPURLResponse` (#3936)

* chore: remove deprecated initializers for `FileLists`, `Headers`, and `HTTPURLResponse`

* fix: fix fixtures

* Plugins 2.0 (#3492)

* Edit CHANGELOG

* Tuist executable (#3494)

* Search for executable in path with tuist prefix if not tuist command

* Add acceptance test for new task

* Update documentation

* Check if executable exists

* Update projects/docs/docs/commands/task.md

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Update Sources/TuistKit/Commands/TuistCommand.swift

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Fix finding command

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Plugins archive (#3531)

Fix build issues

Build and zip executable products

Add tests for buildFatReleaseBinary

Write tests for PluginsArchiveService

Use plugin instead of plugins

Add success print

Format swift code

Update Sources/TuistKit/Services/PluginArchiveService.swift

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Tuist plugins 2.0 fetch (#3577)

* Tuist executable (#3494)

* Search for executable in path with tuist prefix if not tuist command

* Add acceptance test for new task

* Update documentation

* Check if executable exists

* Update projects/docs/docs/commands/task.md

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Update Sources/TuistKit/Commands/TuistCommand.swift

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Fix finding command

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* WIP: Fetch release

* Install tasks to release dir

* Execute plugin task

* Add TuistServiceTests

* Add FetchServiceTests

* Remove DependenciesFetchCommand

* Add fetch plugins test

* Fix PluginServiceTests

* Add test for new logic in PluginServiceTests

* Minor changes

* Revert changes to Package.resolved

* Add missing throws

* Rename gitID to gitReference

* Move to Swift-based APIs

* Fix fetch command

* Uncomment analytics code

* Fix acceptance tests for tuist edit

* Restore Package.resolved

* Fix plugins.feature acceptance test

* Add fetch to tasks.feature

* Add fetch to tasks.feature

* Add fetch to scaffold.feature

* Fix scaffold.feature

* Fix tasks.feature

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* Plugin commands (#3663)

* Add plugin build command

* Add unit tests

* Add plugin acceptance tests

* Update tuist_plugin submodule

* Plugins 2.0: dependencies commands (#3656)

* Fix fetch call when generating via fourier

* Remove clean command exclusive to dependencies

* WIP: tuist update

* Add update flag to tuist fetch

* Reuse code for fetch and update of dependencies

* Fix dependencies update call in acceptance tests

* Format swift code

* Add fetch to tuist focus in fourier

* Add TuistGraph as a dependency of TuistSupportTesting

* Fix fourier tests

* Restructure CleanCategory to mitigate code duplication

* Format swift code

* Lint code

* Update fourier commands

* Add missing ProjectDescription dependency

* Fix CleanServiceTests

* Fix PluginArchiveServiceTests

* Tuist plugins 2.0 ProjectAutomation (#3816)

* WIP: ProjectAutomation for new tasks

* Add swiftToolsSupportDependency to ProjectAutomation

* Make graph method public

* Update tuist_plugin submodule

* Use ENV variable instead of commandline argument for passing tusit binary path

* Read file instead of stdout

* Fix tests

* Remove old tasks and ProjectAutomation

* Fix acceptance tests

* Update error documentation

* Check path in TuistServiceTests

* Format swift code

* Tuist plugin

* Fix ManifestTests

* Fix test_edit_when_there_are_helpers_and_setup_and_config_and_dependencies test

* Add path argument to graph

* Remove plugin zip path if it exists

* Format swift code

* Fix tasks.feature

* Remove log

* Format swift code

* Fix lint issues

* Add docs for tuist task plugins (#3898)

* Add docs for tuist task plugins

* Update docs for fetching dependencies

* Address code review comments

Co-authored-by: Daniele Formichelli <df@bendingspoons.com>

* [3.0] Merge pull request #3949 from tuist/laxmorek/bump-swift-argument-parser

Update `swift-argument-parser`: `.upToNextMajor(from: "0.4.3")` -> `.upToNextMajor(from: "1.0.0")`.

* [3.0] Fixed changelog.

* feat: add `type` parameter to `TargetDependency.sdk` (#3961)

* feat: replace SourceFileGlob init with static method (#3960)

* feat: replace SourceFileGlob init with static method

#393

* fix: glob static method should be added to ProjectDescription

* chore: lint

* fix: compilation error

* fix: fixtures

* fix: fix test compilation error

* fix: test compilation error

* chore: try to fix .build folder caching

* feat: move `disableSynthesizedResourceAccessors` from `Config.swift` to `Project.ProjectOption` (#3963)

* feat: move `disableSynthesizedResourceAccessors` from `Config.swift` to `Project.ProjectOption`

* chore: linting

* fix: fix fixture

* feat: move disableBundleAccessors to ProjectOption

* fix: compilation errors

* chore: fix Project.swift

* chore: formatting

* 3️⃣ Added `sources` property to `ProjectAutomation.Target` model. (#3988)

* [extend-graph-with-sources] Added `sources` property to `ProjectAutomation.Target` model.

* [extend-graph-with-sources] `./fourier lint tuist --fix`

* [extend-graph-with-sources] Updated docs.

* 3️⃣ Drop `tuist lint` command (#4001)

* 3️⃣  Merge `tuist focus` and `tuist generate` command (#3912)

* Remove unused import

* Fix plugins loading for Dependencies.swift

* Fix lint

* Fix unit test

* Update step definition for `tuist fetch`

* fix: fix app_with_spm_dependencies fixturre

Co-authored-by: Marek Fořt <marekfort@me.com>
Co-authored-by: Luis Padron <luis.padron@compass.com>
Co-authored-by: Kamil Harasimowicz <kamil.harasimowicz@gmail.com>
Co-authored-by: Alfredo Delli Bovi <alfredo.dellibovi@gmail.com>
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.

None yet

5 participants