Skip to content

Update README.md#131

Closed
bohdan-ios wants to merge 1 commit intoKolos65:mainfrom
bohdan-ios:patch-2
Closed

Update README.md#131
bohdan-ios wants to merge 1 commit intoKolos65:mainfrom
bohdan-ios:patch-2

Conversation

@bohdan-ios
Copy link
Copy Markdown

I spent hours for debugging xcodebuild on CI, it does not throw an error for not enabling Macros on a machine

Added tips for CI installation.

Added tips for CI installation

* Setting Xcode defaults:
```bash
defaults write com.apple.dt.Xcode IDESkipPackagePluginFingerprintValidatation -bool YES
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Let's remove this line as plugin validation is not related to Mockable.

Copy link
Copy Markdown
Owner

@Kolos65 Kolos65 left a comment

Choose a reason for hiding this comment

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

Pls address review comments.

Read the [installation guide](https://kolos65.github.io/Mockable/documentation/mockable/installation/) of the documentation for more details on how to integrate **Mockable** with your project.

> [!TIP]
> When using the plugin for the first time, be sure to trust and enable
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

let's change "plugin" to "macro" here

@Kolos65
Copy link
Copy Markdown
Owner

Kolos65 commented Dec 8, 2025

Thanks for the PR @bohdan-ios (been there done that btw 😅)

Pls check my comments and change your commit message to use conventional commit message to pass the validation pipeline:
docs: update README.md with CI setup for example.

> it when prompted. If a macros build warning exists, select it to trust
> and enable the macros as well.

For unattended use (e.g. on CI), package plugin and macro validations can be disabled with either of the following:
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Remove package pluginhere as well.

> [!TIP]
> When using the plugin for the first time, be sure to trust and enable
> it when prompted. If a macros build warning exists, select it to trust
> and enable the macros as well.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think we don't need the last part here: "If a macros build warning exists, select it to trust
and enable the macros as well."

@Kolos65
Copy link
Copy Markdown
Owner

Kolos65 commented Dec 10, 2025

Merged in #132

@Kolos65 Kolos65 closed this Dec 10, 2025
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.

2 participants