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

Prepend KEEP ID to documents to keep them sorted #368

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nikitabobko
Copy link
Member

ls output to see how it looks

total 628K
drwxr-xr-x 5 bobko bobko 4.0K Jan  2 17:35 ./
drwxr-xr-x 5 bobko bobko 4.0K Jan  2 17:35 ../
-rw-r--r-- 1 bobko bobko 4.8K Oct 18  2022 android-extensions-entity-caching.md
-rw-r--r-- 1 bobko bobko 3.9K Jan  2 15:47 annotation-instantiation.md
-rw-r--r-- 1 bobko bobko   77 Jan  2 17:35 context-receivers.md
-rw-r--r-- 1 bobko bobko  95K Jan  2 15:47 coroutines.md
-rw-r--r-- 1 bobko bobko   45 Jan  2 17:23 experimental.md
drwxr-xr-x 2 bobko bobko 4.0K Oct 18  2022 extensions/
drwxr-xr-x 2 bobko bobko 4.0K Oct 18  2022 js/
-rw-r--r-- 1 bobko bobko  22K Jan  2 17:35 KEEP-0004-type-aliases.md
-rw-r--r-- 1 bobko bobko  16K Jan  2 17:35 KEEP-0005-bound-callable-references.md
-rw-r--r-- 1 bobko bobko 1.4K Jan  2 17:35 KEEP-0025-local-delegated-properties.md
-rw-r--r-- 1 bobko bobko 3.4K Jan  2 17:35 KEEP-0029-sealed-class-inheritance.md
-rw-r--r-- 1 bobko bobko 8.6K Jan  2 17:35 KEEP-0030-jdk-dependent-built-ins.md
-rw-r--r-- 1 bobko bobko 5.5K Jan  2 17:35 KEEP-0031-data-class-inheritance.md
-rw-r--r-- 1 bobko bobko 4.5K Jan  2 17:35 KEEP-0032-destructuring-in-parameters.md
-rw-r--r-- 1 bobko bobko 1.3K Jan  2 17:35 KEEP-0034-inline-properties.md
-rw-r--r-- 1 bobko bobko 8.3K Jan  2 17:35 KEEP-0045-explicit-api-mode.md
-rw-r--r-- 1 bobko bobko 4.0K Jan  2 17:35 KEEP-0055-underscore-for-unused-parameters.md
-rw-r--r-- 1 bobko bobko 5.2K Jan  2 17:35 KEEP-0057-scope-control-for-implicit-receivers.md
-rw-r--r-- 1 bobko bobko 3.0K Jan  2 17:35 KEEP-0059-underscores-in-numeric-literals.md
-rw-r--r-- 1 bobko bobko 3.9K Jan  2 17:35 KEEP-0061-generic-values-and-valueof-for-enums.md
-rw-r--r-- 1 bobko bobko  43K Jan  2 17:35 KEEP-0075-scripting-support.md
-rw-r--r-- 1 bobko bobko  17K Jan  2 17:35 KEEP-0079-jsr-305-custom-nullability-qualifiers.md
-rw-r--r-- 1 bobko bobko 1.5K Jan  2 17:35 KEEP-0086-local-and-top-level-lateinit-vars.md
-rw-r--r-- 1 bobko bobko  19K Jan  2 17:35 KEEP-0095-opt-in.md
-rw-r--r-- 1 bobko bobko  30K Jan  2 17:35 KEEP-0104-inline-classes.md
-rw-r--r-- 1 bobko bobko 4.9K Jan  2 17:35 KEEP-0107-functional-types-with-big-arity-on-jvm.md
-rw-r--r-- 1 bobko bobko 4.1K Jan  2 17:35 KEEP-0120-val-in-when-subject.md
-rw-r--r-- 1 bobko bobko  19K Jan  2 17:35 KEEP-0135-unsigned-types.md
-rw-r--r-- 1 bobko bobko  19K Jan  2 17:35 KEEP-0139-kotlin-contracts.md
-rw-r--r-- 1 bobko bobko 1015 Jan  2 17:35 KEEP-0150-jvm-static-annotation-in-interface-companion.md
-rw-r--r-- 1 bobko bobko 1.6K Jan  2 17:35 KEEP-0152-jvm-field-annotation-in-interface-companion.md
-rw-r--r-- 1 bobko bobko 7.8K Jan  2 17:35 KEEP-0158-enhancing-main-convention.md
-rw-r--r-- 1 bobko bobko 2.6K Jan  2 17:35 KEEP-0193-named-arguments-in-their-own-position.md
-rw-r--r-- 1 bobko bobko  14K Jan  2 17:35 KEEP-0226-sealed-interface-freedom.md
-rw-r--r-- 1 bobko bobko  10K Jan  2 17:35 KEEP-0233-jvm-records.md
-rw-r--r-- 1 bobko bobko 9.7K Jan  2 17:35 KEEP-0257-repeatable-annotations.md
-rw-r--r-- 1 bobko bobko  61K Jan  2 17:35 KEEP-0259-context-receivers.md
-rw-r--r-- 1 bobko bobko 4.8K Jan  2 17:35 KEEP-0268-definitely-non-nullable-types.md
-rw-r--r-- 1 bobko bobko  11K Jan  2 17:35 KEEP-0283-enum-entries.md
-rw-r--r-- 1 bobko bobko  18K Jan  2 17:35 KEEP-0317-data-objects.md
-rw-r--r-- 1 bobko bobko 6.9K Jan  2 17:35 KEEP-0320-subclass-opt-in-required.md
-rw-r--r-- 1 bobko bobko 8.0K Jan  2 17:35 KEEP-0326-break-continue-in-inline-lambdas.md
-rw-r--r-- 1 bobko bobko 6.7K Jan  2 17:35 KEEP-0328-references-to-java-synthetic-properties.md
-rw-r--r-- 1 bobko bobko 8.8K Oct 18  2022 lateinit-property-isinitialized-intrinsic.md
-rw-r--r-- 1 bobko bobko   77 Jan  2 17:35 scripting-support.md
drwxr-xr-x 2 bobko bobko 4.0K Jan  2 17:35 stdlib/

I changed the names only to realize that links will break :(

Well, links between pull requests and documents in master break each time we merge pull requests

@zarechenskiy it's up to you to decide whether to accept this PR. Feel free to decline the PR without any reasons

My opinion is the following: I think that the current system is already broken (links between blobs in PRs and blobs in master are different anyway), so I think it's ok to break links in master. Links are broken this way or another, but at least we can have the documents nicely sorted, to skim them

I'm going to break link formats anyway in the following commits.
According to https://github.com/Kotlin/KEEP/graphs/traffic this document
isn't that popular
Pros:
- Files are nicely sorted. Now it's easier to take a look at the latest
  proposals

Cons:
- Links are broken because of this commit (I will add redirects to
  popular proposals in the next commit)
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

2 participants