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.

Sign up for GitHub

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

[0.10] Support Smithy IDL Serialization #284

Merged
merged 16 commits into from
Feb 27, 2020
Merged

[0.10] Support Smithy IDL Serialization #284

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
543857c
Add writeInline method to CodeWriter
JordonPhillips Feb 21, 2020
c0b8ce5
Support Smithy IDL serialization
JordonPhillips Feb 21, 2020
e809796
Add filtering predicates to idl serializer
JordonPhillips Feb 24, 2020
675b7d1
Don't filter metadata case-insensitively
JordonPhillips Feb 24, 2020
0f8205a
Serialize metadata-only models
JordonPhillips Feb 24, 2020
6fca073
Clarify documentation on IDL serializer outputs
JordonPhillips Feb 24, 2020
5555a4e
Make IDL shape comparator private
JordonPhillips Feb 24, 2020
9397ca4
Sort structures before unions in serialized IDL
JordonPhillips Feb 24, 2020
496f43a
touch up docs
JordonPhillips Feb 24, 2020
a96403a
Make shapeplacer public
JordonPhillips Feb 24, 2020
9ef38f4
Privatize everything that can be
JordonPhillips Feb 24, 2020
dd4396b
Use string array for writeInline
JordonPhillips Feb 24, 2020
b57193f
remove unused import
JordonPhillips Feb 25, 2020
8ea2e2f
Simplify IDL serializer
JordonPhillips Feb 27, 2020
92a4eaa
Add a base path to the idl serializer
JordonPhillips Feb 27, 2020
1811d0a
Compare ShapeId's case-insensitively
JordonPhillips Feb 27, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Don't filter metadata case-insensitively 
Shapes can be filtered case-insensitively because they are required
to be case-insensitively unique. Metadata doesn't have that same
constraint though, so filtering them as such would lead to a non-
deterministic ordering.
  • Loading branch information
@JordonPhillips
JordonPhillips committed Feb 24, 2020
commit 675b7d12d1a2ec9da32c5f2fb74f4c951ae341d9
2 changes: 1 addition & 1 deletion smithy-model/src/main/java/software/amazon/smithy/model/shapes/SmithyIdlModelSerializer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ private String serializeHeader(String namespace, Model fullModel) {
// Write the full metadata into every output. When loaded back together the conflicts will be ignored,
// but if they're separated out then each file will still have all the context.
fullModel.getMetadata().entrySet().stream()
.sorted(Map.Entry.comparingByKey(String.CASE_INSENSITIVE_ORDER))
.sorted(Map.Entry.comparingByKey())
.filter(entry -> metadataFilter.test(entry.getKey()))
JordonPhillips marked this conversation as resolved.
Show resolved Hide resolved
.forEach(entry -> {
codeWriter.trimTrailingSpaces(false)
Expand Down
2 changes: 2 additions & 0 deletions ...est/resources/software/amazon/smithy/model/shapes/idl-serialization/cases/metadata.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
$version: "0.5.0"

metadata CaseSensitive = true
metadata caseSensitive = true
metadata example.array = [
10,
true,
Expand Down