-
Notifications
You must be signed in to change notification settings - Fork 1.6k
RFC: Packages as (optional) namespaces #3243
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
e590c72
Copy over RFC text from https://github.com/Manishearth/namespacing-rfc/
Manishearth 5e6ad7c
Fix template
Manishearth d9d1556
Add basic FAQ
Manishearth e29aa55
Add more separator options
Manishearth 3a0b099
leaf crate names
Manishearth 4422adf
sep choice
Manishearth 24db227
Clarify motivation on projects vs organizations
Manishearth 1986c3d
Switch to colons
Manishearth 42046f8
fixes
Manishearth 2fff101
Document Python prior art
epage 8e667ce
Expand on the guide-level explanation
epage 01d37b2
Document feature flags drawbacks
epage 4501047
Merge pull request #4 from epage/guide
Manishearth 1f62468
Merge pull request #3 from epage/prior
Manishearth 1ca3e62
Merge pull request #5 from epage/flags
Manishearth f7be349
Fix a couple of typos
epage 9a144fb
Updaet motivation for `::` semantics
epage a578a67
Merge pull request #7 from epage/motivation
Manishearth 3396a2e
Merge pull request #6 from epage/typos
Manishearth 11c6354
Update summary to focus on `::` semantics
epage cc5872d
Merge pull request #8 from epage/summary
Manishearth 65f7083
add renames as unresolved q
Manishearth 5c497c5
fix syntax
Manishearth 67ef913
fixes
Manishearth 56fae2e
trie-unresolved
Manishearth 482fae8
Update 0000-packages-as-optional-namespaces.md
Manishearth 4c14f9c
Update text/0000-packages-as-optional-namespaces.md
Manishearth d9a3c90
Update text/0000-packages-as-optional-namespaces.md
Manishearth 8bd7fbc
Add .crate file as unresolved
epage abde93f
Be explicit that open questions are deferred out
epage 7d77485
Call out distributions
epage 75dd867
Merge pull request #9 from epage/unresolved
Manishearth 0db096f
Update text/0000-packages-as-optional-namespaces.md
Manishearth 6cc8886
Rename 0000-packages-as-optional-namespaces.md to 3243-packages-as-op…
oli-obk a93b1e4
Add RFC PR link
oli-obk 656203f
Link to rust tracking issue
oli-obk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add RFC PR link
- Loading branch information
commit a93b1e4fb81cf6157b1fd5c4ff7a2f1fe50fef7c
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you forget to change this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed via #3586