Skip to content

Commit c99ec59

Browse files
committed
fixed remark-lint issues
there were 2 obsolete plugins that needed to be removed
1 parent 4255a15 commit c99ec59

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+428
-258
lines changed

.github/PULL_REQUEST_TEMPLATE.md

+18-5
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,43 @@
11
<!--- Provide a general summary of your changes in the Title above -->
22

33
## Description
4+
45
<!--- Describe your changes in detail -->
56

67
## Related Issue
8+
79
<!--- This project only accepts pull requests related to open issues -->
10+
811
<!--- If suggesting a new feature or change, please discuss it in an issue first -->
12+
913
<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->
14+
1015
<!--- Please link to the issue here: -->
1116

1217
## Motivation and Context
18+
1319
<!--- Why is this change required? What problem does it solve? -->
1420

1521
## How Has This Been Tested?
22+
1623
<!--- Please describe in detail how you tested your changes. -->
24+
1725
<!--- Include details of your testing environment, and the tests you ran to -->
26+
1827
<!--- see how your change affects other areas of the code, etc. -->
1928

2029
## Screenshots (if appropriate):
2130

31+
<!--- Drag and drop screenshots here -->
32+
2233
## Checklist:
34+
2335
<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
36+
2437
<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
2538

26-
- [ ] My code follows the code style of this project.
27-
- [ ] My change requires a change to the documentation.
28-
- [ ] I have updated the documentation accordingly.
29-
- [ ] I have added tests to cover my changes.
30-
- [ ] All new and existing tests passed.
39+
* \[ ] My code follows the code style of this project.
40+
* \[ ] My change requires a change to the documentation.
41+
* \[ ] I have updated the documentation accordingly.
42+
* \[ ] I have added tests to cover my changes.
43+
* \[ ] All new and existing tests passed.

.remarkrc.yaml

-2
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ plugins:
2121
- remark-lint-list-item-bullet-indent
2222
- remark-lint-list-item-content-indent
2323
- remark-lint-maximum-heading-length: 120
24-
- remark-lint-no-auto-link-without-protocol
2524
- remark-lint-no-blockquote-without-marker
2625
- remark-lint-no-consecutive-blank-lines
2726
- remark-lint-no-duplicate-definitions
@@ -36,7 +35,6 @@ plugins:
3635
- remark-lint-no-heading-content-indent
3736
- remark-lint-no-heading-indent
3837
- remark-lint-no-heading-like-paragraph
39-
- remark-lint-no-inline-padding
4038
- remark-lint-no-literal-urls
4139
- remark-lint-no-multiple-toplevel-headings
4240
- remark-lint-no-reference-like-url

BREAKING_CHANGES.md

+41-12
Original file line numberDiff line numberDiff line change
@@ -11,64 +11,91 @@
1111
### Configuration changes:
1212

1313
* The configuration properties `continuous-delivery-fallback-tag`, `tag-number-pattern`, and `tag` were renamed to `continuous-delivery-fallback-label`, `label-number-pattern`, and `label` respectively. `tag-pre-release-weight` and `tag-prefix` remained as they were as they are referring to a Git tag.
14+
1415
* When using a commit message that matches **both** `*-version-bump-message` and `no-bump-message`, there is no increment for that commit. In other words, `no-bump-message` now takes precedence over `*-version-bump-message`.
16+
1517
* The fallback version strategy now returns `0.0.0` and is flagged with `ShouldIncrement` equal to `true`. This yields the version `0.1.0` on the `develop` branch (`IncrementStrategy.Minor` by default) and `0.0.1` on the `main` branch (`IncremetnStrategy.Patch` by default).
18+
1619
* The current branch (child) inherits its configuration from the source (parent) branch if the `increment` strategy is set to `Inherit`. This makes branch configuration recursive, simpler, more intuitive, more flexible, and more robust.
20+
1721
* Instead of having a single effective configuration, we now have one effective configuration per branch where the increment strategy is not set to `inherit`.
22+
1823
* The new implementation of the branch configuration inheritance affects per default only the pull-requests, hotfix and feature branches. In this case the next version will be generated like the child branch is not existing and the commits have been made on the source branch.
1924
* The following example illustrates this behavior. On the feature branch the semantic version `1.1.0-just-a-test.1+2` will now be generated instead of version `1.0.0-just-a-test.1+3` previously:
2025

21-
``` log
26+
```log
2227
* 1f1cfb4 52 minutes ago (HEAD -> feature/just-a-test)
2328
* 1f9654d 54 minutes ago (release/1.1.0)
2429
* be72411 56 minutes ago (develop)
2530
* 14800ff 58 minutes ago (tag: 1.0.0, main)
2631
```
2732
2833
* A new `unknown` branch magic string has been introduced to give the user the possibility to specify the branch configuration for a branch which is not known. A branch is not known if only the regular expression of the branch configuration with the name `unknown` is matching. Please notice that this branch configuration behaves like any other branch configurations.
34+
2935
* Additional `fallback` branch configuration properties have been introduced at the root to define base properties which will be inherit to the branch configurations. That means if no other branch configuration in the inheritance line defines the given property the fallback property applies. Notice that the inheritance tree can be controlled using the increment strategy property in the branch configuration section.
3036
* The following example illustrates this behavior. The hotfix branch configuration overrides the main branch configuration and the result overrides the fallback branch configuration.
3137
32-
``` log
38+
```log
3339
* 1f1cfb4 52 minutes ago (HEAD -> hotfix/just-a-test)
3440
* 14800ff 58 minutes ago (tag: 1.0.0, main)
3541
```
3642
3743
* When overriding the configuration with e.g. GitVersion.yaml the software distinguishes between properties who are not existent and properties who are `null`. This is especially important if the user wants to define branch related configuration which are marked with `increment` strategy `Inherit`.
44+
3845
* Following root configuration properties have been removed:
3946
* continuous-delivery-fallback-tag
47+
4048
* A new branch related property with name `track-merge-message` has been introduced. Consider we have a `main` branch and a `release/1.0.0` branch and merge changes from `release/1.0.0` to the main branch. In this scenario the merge message will be interpreted as a next version `1.0.0` when `track-merge-message` is set to `true` otherwise `0.0.1`.
49+
4150
* The pre-release tags are only considered when they are matching with the label name of the branch. This has an effect on the way how the `CommitCountSource` will be determined.
51+
4252
* The process of increasing the version with bump message when `CommitMessageIncrementing` is enabled and increment strategy is `None` has been changed.
53+
4354
* A new configuration property with name `version-in-branch-pattern` has been introduced. This setting only applies on branches where the option `is-release-branch` is set to `true`. Please notice that the branch name needs to be defined after the version number by default (instead of `support/lts-2.0.0` please name the branch like `support/2.0.0-lts`).
55+
4456
* The `is-release-branch` property of the `hotfix` branch setting has been changed from `false` to `true`. If present the hotfix number will be considered now by default.
57+
4558
* In the GitHub and the Git Flow workflows the `label` property is by default set to an empty string on the `main` branch. This yields to a pre-release version on `main` with an empty tag. Instead of for instance `1.0.1+46` GitVersion generates the full semantic version `1.0.1-46` instead. This behavior can be changed to generate only stable versions (no pre-release version) with setting the label to `null` (Please keep in mind that the `label` property on root needs to be set to `null` as well, otherwise the fallback applies). This change is caused by issue #2347.
59+
4660
* The `useBranchName` magic string has been removed. Instead use `{BranchName}` for `label`.
61+
4762
* The `BranchPrefixToTrim` configuration property has been removed. `RegularExpression` is now used to capture named groups instead.
4863
* Default `RegularExpression` for feature branches is changed from `^features?[/-]` to `^features?[/-](?<BranchName>.+)` to support using `{BranchName}` out-of-the-box
4964
* Default `RegularExpression` for unknown branches is changed from `.*` to `(?<BranchName>.+)` to support using `{BranchName}` out-of-the-box
65+
5066
* The `Mainline` mode and the related implementation has been removed completely. The new `Mainline` version strategy should be used instead.
67+
5168
* The `Mainline` version strategy doesn't support downgrading the increment for calculating the next version. This is the case if e.g. a bump messages has been defined which is lower than the branch increment.
69+
5270
* The branch related property `is-mainline` in the configuration system has been renamed to `is-main-branch`
71+
5372
* The versioning mode has been renamed to deployment mode and consists of following values:
5473
* ManualDeployment (previously ContinuousDelivery)
5574
* ContinuousDelivery (previously ContinuousDeployment)
5675
* ContinuousDeployment (new)
76+
5777
* At the configuration root level, a new array called `strategies` has been introduced, which can consist of on or more following values:
5878
* ConfiguredNextVersion
5979
* MergeMessage
6080
* TaggedCommit
6181
* TrackReleaseBranches
6282
* VersionInBranchName
6383
* Mainline
84+
6485
* The initialization wizard has been removed.
86+
6587
* On the `develop`, `release` and `hotfix` branch the introduced branch related property `prevent-increment.when-current-commit-tagged` has been set to `false` to get the incremented instead of the tagged semantic version.
88+
6689
* When setting the "ignore commits before" parameter to a future value, an exception will occur if no commits are found on the current branch. This behavior mimics that of an empty repository.
90+
6791
* On the `GitFlow` workflow the increment property has been changed:
6892
* in branch `release` from `None` to `Minor` and
6993
* in branch `hotfix` from `None` to `Patch`
94+
7095
* On the `GitHubFlow` workflow the increment property has been changed in branch `release` from `None` to `Patch`.
96+
7197
* When creating a branch with name `hotfix/next` (by using the `GitFlow` workflow) or `release/next` (by the `GitHubFlow` workflow) the resulting version will yield to a patched version per default.
98+
7299
* If you have a tag `1.0.0` on `main` and branch from `main` to `release/1.0.1` then the next version number will be `1.1.0` when using the `GitFlow` workflow. This behavior is expected (but different compared to the `GitHubFlow` workflow) because on the `GitFlow` workflow you have an addition branch configuration with name hotfix where `is-release-branch` is set to `true`. That means if you want `1.0.1` as a next version you need to branch to `hotfix/1.0.1` or `hotfix/next`. On the other hand if you use the `GitHubFlow` workflow the next version number will be `1.0.1` because the increment on the `release` branch is set to `Patch`.
73100
74101
### Legacy Output Variables
@@ -87,9 +114,9 @@ The following legacy output variables have been removed in this version:
87114
## v5.0.0
88115
89116
* Version numbers in branches other than `release` branches are no longer
90-
considered as a version source by default. Implemented in [#1541][pr-1541].
117+
considered as a version source by default. Implemented in [#1541][pr-1541].
91118
* [#1581][pr-1581] folds `GitTools.Core` back into GitVersion to make
92-
maintaining GitVersion easier.
119+
maintaining GitVersion easier.
93120
94121
## v4.0.0
95122
@@ -107,27 +134,29 @@ work for you
107134
* `GitVersionConfig.yaml` is deprecated in favor of `GitVersion.yml`.
108135
* Regular expressions are no longer used as keys in branch config
109136
* We have named branches, and introduced a `regex` config which you can
110-
override.
137+
override.
111138
* The default keys are: `master`, `develop`, `feature`, `release`, `pull-request`,
112-
`hotfix` and `support`
139+
`hotfix` and `support`
113140
* Just run `GitVersion.exe` in your project directory and it will tell you
114-
what to change your config keys to
141+
what to change your config keys to
115142
* For example, `dev(elop)?(ment)?$` is now just `develop`, we suggest not
116-
overring regular expressions unless you really want to use a different convention.
143+
overring regular expressions unless you really want to use a different convention.
117144
* `source-branches` added as a configuration option for branches, it helps
118-
GitVersion pick the correct source branch
145+
GitVersion pick the correct source branch
119146
120147
## v3.0.0
121148
122149
* NextVersion.txt has been deprecated, only `GitVersionConfig.yaml` is supported
123150
* `AssemblyFileSemVer` variable removed, `AssemblyVersioningScheme` configuration
124-
value makes this variable obsolete
151+
value makes this variable obsolete
125152
* Variables `ClassicVersion` and `ClassicVersionWithTag` removed
126153
* MSBuild task arguments (`AssemblyVersioningScheme`, `DevelopBranchTag`,
127-
`ReleaseBranchTag`, `TagPrefix`, `NextVersion`) have been removed, use
128-
`GitVersionConfig.yaml` instead
154+
`ReleaseBranchTag`, `TagPrefix`, `NextVersion`) have been removed, use
155+
`GitVersionConfig.yaml` instead
129156
* GitVersionTask's `ReleaseDateAttribute` no longer exists
130157
131158
[gitflow]: https://gitversion.net/docs/learn/branching-strategies/gitflow-examples_complete
159+
132160
[pr-1541]: https://github.com/GitTools/GitVersion/pull/1541
161+
133162
[pr-1581]: https://github.com/GitTools/GitVersion/pull/1581

CONTRIBUTING.md

+6-6
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@ Issues are also welcome, [failing tests](#writing-tests) are even more welcome.
1717
* Try to use feature branches rather than developing on main.
1818
* Please include tests covering the change.
1919
* The documentation is stored in the repository under the [`docs`](docs) folder.
20-
Have a look at the [documentation readme file](docs/readme.md) for guidance
21-
on how to improve the documentation and please include documentation updates
22-
with your PR.
20+
Have a look at the [documentation readme file](docs/readme.md) for guidance
21+
on how to improve the documentation and please include documentation updates
22+
with your PR.
2323

2424
## How it works
2525

@@ -100,15 +100,15 @@ We use Cake for our build and deployment process. The way the release process is
100100

101101
1. We build releasable artifacts with GitHub Actions
102102
2. We create a milestone for the release if it's not already created. Our milestones are named using the semver.
103-
For example `5.12.0` or `6.0.0-beta.2`
103+
For example `5.12.0` or `6.0.0-beta.2`
104104
3. We move all the closed issues and closed pull requests that are going to be included in the release to the milestone.
105105
4. We check that all the issues and pull requests that are going to be included in the release have a label assigned,
106-
otherwise it will fail the release.
106+
otherwise it will fail the release.
107107
5. We create a release in the GitHub UI, and create a tag and name it using the milestone name. For example `5.12.0` or `6.0.0-beta.2`
108108
6. We specify if the release is a pre-release or latest release in the GitHub UI.
109109
7. We publish the release.
110110
8. The GitHub Actions will create a GitHub release and publish the artifacts to NuGet, Chocolatey, Docker, Homebrew
111-
and other distribution channels.
111+
and other distribution channels.
112112
9. The issues and pull requests will get updated with message specifying in which release it was included.
113113

114114
## Code Style

0 commit comments

Comments
 (0)