This repository has been archived by the owner on Jul 15, 2023. It is now read-only.
Releases from master branch #869
Labels
Domain: Releases
Scheduling and infrastructure around package releases.
Domain: Tooling
Repository tasks around improving source tooling.
Status: In Discussion
Please continue discussing the proposed change before sending a pull request.
Feature request
Is your feature request that we implement a new rule?
No
Is your feature request related to a problem? Please describe.
Current release process that requires to keep second repo copy for
release
branch and to manually copy files fromdist/build
to root ofrelease
branch is annoying and error prone (there is diff between folders already).Currently there are differences in contents of
dist/build
andrelease
branch:Worth to note that
tslint.json
is absent indist/build
but will be used when users add"extends": ["tslint-microsoft-contrib"]
to theirtslint.json
as shown in docs.Also it has diff with
legacy.json
that we expect it should alias:Basically
legacy.json
replaced deprecated rules with new from core, disabledno-reserved-keywords
and has 2 new rules.Describe the solution you'd like
Release packages directly from
dist/build
folder, this will enable #863. Alsosemantic-release
supports releasing from specified directory (seepkgRoot
)If I understand correctly - removing files from package can be considered as breaking change and should be major version increment. To avoid this I propose to create folder with these files and copy them to
dist/build
during package build. This step will be removed in next major version.As for
tslint.json
- two options possible:void-zero
,react-a11y-accessible-headings
fromlegacy.json
and copylegacy.json
astslint.json
tobuild/dist
(OK if they have proper TSLint version, notification if they have old TSLint version).Describe alternatives you've considered
#825 will be a breaking change due to changes in package structure while this change to build steps may be preliminary step.
Alternatively - #825 may be closed in favor of this
Additional context
N/A
The text was updated successfully, but these errors were encountered: