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

feat: add feature resolution for protobuf editions #2029

Merged
merged 45 commits into from
Oct 16, 2024
Merged
Show file tree
Hide file tree
Changes from 16 commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
b4b5ca4
feat: api_converters_editions tests added and run successfully"
sofisl Aug 23, 2024
d8eb1b4
remove logging
sofisl Aug 26, 2024
65d3ed1
feat: add feature resolution for protobuf editions
sofisl Sep 10, 2024
0a7bbd1
cleanup PR
sofisl Sep 10, 2024
68e4480
see diff
sofisl Sep 10, 2024
759cc45
fix broken tests
sofisl Sep 10, 2024
8dabd5e
add tests
sofisl Sep 10, 2024
68b5339
feat: add feature resolution and tests
sofisl Sep 14, 2024
dae1b3e
undo lockfile update
sofisl Sep 14, 2024
f204288
add comment
sofisl Sep 14, 2024
f0491be
cleanup
sofisl Sep 14, 2024
d9c2ec4
remove only
sofisl Sep 16, 2024
b802c42
save for now
sofisl Sep 17, 2024
73590cc
undo bundle changes
sofisl Sep 17, 2024
a70c6a4
revert package-lock changes
sofisl Sep 17, 2024
a9ffc8a
feat: add feature resolution
sofisl Sep 19, 2024
43a703a
revert package-lock updates/
sofisl Sep 19, 2024
2fabe04
cleanup
sofisl Sep 19, 2024
1bd72ff
chore: run lint
sofisl Sep 19, 2024
5d35287
Merge branch 'master' into protobufEditionsWork
sofisl Sep 19, 2024
225fddb
typo
sofisl Sep 19, 2024
840907e
Merge branch 'protobufEditionsWork' of github.com:protobufjs/protobuf…
sofisl Sep 19, 2024
43183bd
typo
sofisl Sep 19, 2024
c4c486d
Update api_enum.js
sofisl Sep 20, 2024
1ca7586
respond to comments
sofisl Sep 24, 2024
e73bb3c
chore: add grammar tests
sofisl Sep 24, 2024
e1abb31
run lint
sofisl Sep 24, 2024
10dae84
address comments
sofisl Sep 24, 2024
087a33e
remove comment
sofisl Sep 24, 2024
608009b
respond to comments
sofisl Sep 26, 2024
062caa5
fix test
sofisl Sep 26, 2024
c7cee06
add default feature resolution at the end
sofisl Oct 2, 2024
da2df8a
run lint
sofisl Oct 2, 2024
c11ddf4
Update object.js
sofisl Oct 2, 2024
d2d47d9
fix: change tree traversal order and feature resolution algorithm
sofisl Oct 4, 2024
2d81627
resolve conflicts
sofisl Oct 4, 2024
b57bde7
run lint
sofisl Oct 4, 2024
da2663a
Update feature_resolution_editions.js
sofisl Oct 4, 2024
caad500
tests: added and broke up tests
sofisl Oct 8, 2024
1f1d33c
chore: remove specific overwriting
sofisl Oct 8, 2024
4ca95c3
chore: respond to test comments
sofisl Oct 9, 2024
a9eebac
simplify tests
sofisl Oct 9, 2024
6f7d540
chore: respond to comments
sofisl Oct 14, 2024
530d59a
chore: respond to comments
sofisl Oct 15, 2024
23c8ab9
respond to comments
sofisl Oct 16, 2024
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
870 changes: 86 additions & 784 deletions cli/package-lock.json

Large diffs are not rendered by default.

8 changes: 7 additions & 1 deletion index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,9 @@ export class Enum extends ReflectionObject {
/** Values options, if any */
public valuesOptions?: { [k: string]: { [k: string]: any } };

/** Values features, if any */
public _valuesFeatures?: { [k: string]: { [k: string]: any } };

/** Reserved ranges, if any. */
public reserved: (number[]|string)[];

Expand Down Expand Up @@ -877,6 +880,9 @@ export abstract class ReflectionObject {
/** Unique name within its namespace. */
public name: string;

/** Resolved Features. */
public _features: any;

/** Parent namespace. */
public parent: (Namespace|null);

Expand Down Expand Up @@ -1255,7 +1261,7 @@ export class Root extends NamespaceBase {

/**
* Loads a namespace descriptor into a root namespace.
* @param json Nameespace descriptor
* @param json Namespace descriptor
* @param [root] Root namespace, defaults to create a new one if omitted
* @returns Root namespace
*/
Expand Down
Loading