-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix: codegen changes for calling aws sdk from local service #538
base: rishav-awssdkFromLocal-Testmodel
Are you sure you want to change the base?
Changes from 247 commits
b6153fc
60536c2
5937497
fb78c09
18f0711
0adee61
fcb2ecf
0030cd3
5dd7d6f
9e0542f
5db5cfe
3bf0b36
1a5b4b5
4bbf2a6
a92f50c
06d7889
32f14f5
473be0f
478a2e1
2b598e7
56cd0eb
76840fa
eb832a8
e8ce1f4
2615a73
20199da
93684bb
0b0f1ed
4ac299e
327746c
992c669
31ab1d9
8769cee
046b73c
38aae8e
7791bae
6a4e630
72b4384
4bd8251
a4431a7
a2cf766
0e8c581
d37b806
2af2588
a4cb20b
be7301c
db0e27a
712cbcf
ec09dc1
7783d4e
ed8eca6
1d832a8
b7ace1c
ba93f86
51ca930
d4b7c82
9a8e588
f362bb0
fa53236
015825f
355d74a
aee93ba
9a452e2
e92d40c
99df7d7
401c64d
0d4547e
ae6eecb
de202ba
bb25f0a
78cca65
1870322
7ce1847
c8883fb
dd83e91
94afffe
013b2dd
d1b6f1f
99c98a9
e7c8cde
3ca8c6f
aaa0ddd
c23a6cf
5779b87
b910eeb
31fabcf
f1e5936
d5291a3
1226428
b7eabcd
417f9be
dc3f5fa
f0e08c7
135309b
ec28625
622071e
0c6ff38
a21c2ef
4a70ece
0b85561
99ce8d5
8155e77
f176be6
c00a978
60ad10c
be6c61b
e093887
12e2aed
477e6ef
84aec71
b2d454b
a285f84
3601472
1241dc0
0a37314
5bd09e1
dbbca19
00b06ad
2ea8dbf
87260b7
d8977b4
eab1449
80d8381
3a7ed5d
7bac1de
d88e9d0
03cc3f5
89c6054
2fc55a4
4274db5
152922b
bd2619d
c00fd79
dd0a5db
368f395
aa207bb
c51bed4
0b6e0ba
842d5da
1cf85e6
e1099df
9631e0b
4e535fd
64f2109
9bcd8cc
3163256
d3fa74f
2faf98d
6f5bb33
ce6c4cb
012ec0b
1e3ca27
424f689
67f8f03
5280f59
5c31d4a
f8031b5
e61d71f
111ca7e
841c7f3
bb01ef8
7969a94
86a755f
b270c53
cedd2c2
0135c25
ee3fd1c
6c31fb6
b186a1d
b34be61
c124806
5afef0f
11823a9
bf415a4
82ce455
23148b6
95a1eb5
d46203c
13f0c65
e71ef6e
e15c0a0
93d7085
8a4021d
e8804ed
af8a690
1a5c6dd
29e456e
ca2d447
dd055c6
02cf7c5
f7ce0ab
8461540
e2913ab
5765911
749d216
8de40a1
3d1059c
ec7791b
f091075
72d425a
7679bb9
8d03d82
d63c923
1f82a5d
d249234
fb1752d
67d36cd
994adfa
dee1525
e5e2b17
c154ce2
26bfca6
c636ea5
b301c39
c55a347
b509f67
2cc9112
2a5e6e2
0f167f4
36e48f3
cfafd7b
5c96987
2fae3a5
dfacd4c
b122673
5171357
658a854
0294d5b
0a204bf
a1fb3ae
c7ef196
f76fc10
c1ee6ab
60db6e3
83b721b
dd656a6
6a10e5e
39c526e
e5bd697
ce4967a
7f09065
a2150b2
e1a2026
d00ec0b
f887855
036b05f
cf2096f
9107136
1a6f1c9
c1bf94f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -95,7 +95,7 @@ public final class SmithyGoDependency { | |
public static final GoDependency GO_JMESPATH = goJmespath(null); | ||
public static final GoDependency MATH = stdlib("math"); | ||
|
||
private static final String SMITHY_SOURCE_PATH = "github.com/aws/smithy-go"; | ||
public static final String SMITHY_SOURCE_PATH = "github.com/aws/smithy-go"; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We have used public static variable in a lot of places and smithy dafny will not be in production code. So, I made it public instead of having a getter method. |
||
private static final String GO_CMP_SOURCE_PATH = "github.com/google/go-cmp"; | ||
private static final String GO_JMESPATH_SOURCE_PATH = | ||
"github.com/jmespath/go-jmespath"; | ||
|
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.
The problem was this was opaque and this could be created by KMS or DDB in this model as the smithy model has both KMS and DDB
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.
I assume this is the
AcessDeniedException
?EDIT: Maybe in a follow up PR we assert that we do end up with non-opaque exceptions?