Commit 80ff46f
authored
Stop pinning deps in working/macros/example/pubspec.yaml. (#3595)
Pinning the deps was creating a version skew problem in continuous
integration (which is configured in `.github/workflows/dart.yml`). The
continuous integration configuration uses `sdk: main` (meaning that
the code will be analyzed using the latest bleeding edge build of
Dart). But the macros in `working/macros/example` import macro support
functionality from `package:_fe_analyzer_shared/src`, so they use
whatever version of that package that is configured in the pubspec
(which, prior to this change, was pinned to version `3.4.0-56.0.dev`).
The reason this creates a version skew problem is that the macro
support functionality in `package:_fe_analyzer_shared/src` includes
deserialization logic that decodes data structures created during
analysis and compilation. Which means that any time the serialization
format changes, that could potentially break the continuous
integration bot.
Eventually we plan to move the macro support functionality into a
`dart:` library that will ship as part of the SDK, so there will be no
more version skew problem. Until then, an easy way to work around the
version skew is simply to pin the pubspec to the `main` branch rather
than a specific release, so that it matches what's configured in
continuous integration.
A side effect of this change is that if the API for writing macros
gets changed in a way that breaks the examples in
`working/macros/example`, that will cause the continuous integration
bot to immediately start failing. (Previously it wouldn't cause a
failure because the API was pinned to `3.4.0-56.0.dev`). Considering
that the API for macros is under active development, this is probably
a good thing; it will mean that if we make inadvertent API changes
that break the examples, we'll be more likely to find out about it.1 parent aad1953 commit 80ff46f
1 file changed
+17
-17
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
| 25 | + | |
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
30 | | - | |
| 30 | + | |
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
| 35 | + | |
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
| 40 | + | |
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
45 | | - | |
| 45 | + | |
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
50 | | - | |
| 50 | + | |
51 | 51 | | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
55 | | - | |
| 55 | + | |
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
60 | | - | |
| 60 | + | |
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
65 | | - | |
| 65 | + | |
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
70 | | - | |
| 70 | + | |
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
74 | 74 | | |
75 | | - | |
| 75 | + | |
76 | 76 | | |
77 | 77 | | |
78 | 78 | | |
79 | 79 | | |
80 | | - | |
| 80 | + | |
81 | 81 | | |
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
85 | | - | |
| 85 | + | |
86 | 86 | | |
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
90 | | - | |
| 90 | + | |
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
94 | 94 | | |
95 | | - | |
| 95 | + | |
96 | 96 | | |
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
100 | | - | |
| 100 | + | |
0 commit comments