merge discovery common #132

kevmoo · 2021-01-22T23:36:40Z

Initial commit
First version of the common discovery APIs package.
Prepare for publish.
Expose the Escaper class which is also used by the generated library code.
Add DetailedApiRequestError and mapping from JSON
Publish new version of the discovery_commons package.
fix strong mode issues
Rev the pubspec version to capture DDC fixes (fix strong mode issues dart-archive/discoveryapis_commons#5).
Remove crypto dependency and upgrade dart sdk dependency
move to pkg/test
dartfmt
fix imports
Upgrade version in pubspec.yaml
Make everything strong-mode clean (Remove crypto #8)
Prepare for new version, finish strong-mode cleaning and add 'analysis_options.yaml' file
Fix USES_DYNAMIC_AS_BOTTOM violations (Datastore beta3? #10)
Updates for Dart 2.0 core library changes (wave 2.2)
add travis-ci support
Fix strong mode issue with Dart2 (Sheets API V4 - To Be Added? #13)
Fix screaming consts. (timeMin and timMax parameters cause a Bad Request #14)
Fix range check. (Dart googleapis 0.35.0 - Google Drive API v3 - fields request parameter #15)
Dart 2 fix (Is there anyone that monitors these issues? #16)
Prepare release. (Translate API not working correctly #17)
Fix test in Dart 2 mode. (Any Dart-specific documentation? #19)
dartfmt
test: cleanup deprecated usages
analysis: disable implicit casts, enable other lints
Prepare to release 0.1.6+1
misc: fix analyzer error regarding missing return value (Api-Roll 55: 2017-10-05 #21)
Allow string error codes in error responses
Save jsonResponse in exception
Address review
Fix responses where the content-length isn't given (Api-Roll 57: 2017-11-14 #24)
Support latest pkg:http, prepare for release (Failed assertion when doing clientViaUserConsent #26)
Fix new analyzer errors
Enable and fix lints via dartfmt --fix
fix some spelling error
dartfmt fixes
Remove forbidden headers
Send client library identifier
Prepare a new release
Ignore uri not exists
Make ApiRequestError implement Exception and not Error.
Update pubspec.yaml for null safety.
Migrate to null safety.
Update CHANGEOG.md, pubspec.yaml, and .travis.yaml.
Fix lints and enable CI

[email protected], [email protected] BUG= Review URL: https://codereview.chromium.org/991823002

[email protected], [email protected] BUG= Review URL: https://codereview.chromium.org/992893002

…code. [email protected], [email protected] BUG= Review URL: https://codereview.chromium.org/997483004

This commit also adds the necessary tests for this functionality

Add DetailedApiRequestError and mapping from JSON

fix strong mode issues

…pis_commons#5). BUG= [email protected] Review URL: https://codereview.chromium.org/1397293002 .

* Make everything strong-mode clean. Fix deprecation warnings for expectAsync. Make convert parameter for mapMap non-optional. * Run dartfmt

…s_options.yaml' file [email protected] Review-Url: https://codereview.chromium.org/2905613002 .

Updates for Dart 2.0 core library changes (wave 2.2)

They're deprecated in the latest dev SDK, making Travis unhappy. Updated SDK constraint, .cast() was introduced in dev.22.0.

Start and end are both inclusive in ByteRange, so it should be possible to create a range of 0-0. Fixes #6.

Remove forbidden headers

Send client library identifier

Prepare a new release

Ignore uri not exists

The `dart:core` documentation clearly states errors represents a program failure that the programmer should have avoided. However, we throw a `DetailedApiRequestError` instance if the server responds 4xx/5xx.

Make `ApiRequestError` implement `Exception` and not `Error`.

Migrate to null safety

google-cla · 2021-01-22T23:37:14Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

kevmoo · 2021-01-22T23:38:40Z

FYI: the history is ugly here because I merged in the history from the other repo

google-cla · 2021-01-22T23:43:54Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

kevmoo · 2021-01-23T01:37:10Z

Doing the full history is a bit of a mess. Doing something else!

wibling and others added 30 commits March 9, 2015 12:30

Initial commit

64c83aa

First version of the common discovery APIs package.

8d216eb

[email protected], [email protected] BUG= Review URL: https://codereview.chromium.org/991823002

Prepare for publish.

66909e2

[email protected], [email protected] BUG= Review URL: https://codereview.chromium.org/992893002

Expose the Escaper class which is also used by the generated library …

dc125fd

…code. [email protected], [email protected] BUG= Review URL: https://codereview.chromium.org/997483004

Add DetailedApiRequestError and mapping from JSON

ad5aed8

This commit also adds the necessary tests for this functionality

Merge pull request #4 from enyo/DetailedApiRequestError

3b76210

Add DetailedApiRequestError and mapping from JSON

Publish new version of the discovery_commons package.

f98ff31

fix strong mode issues

c6005fe

Merge pull request #5 from dart-lang/strong_mode

6cfb933

fix strong mode issues

Rev the pubspec version to capture DDC fixes (dart-archive/discoverya…

a46c049

…pis_commons#5). BUG= [email protected] Review URL: https://codereview.chromium.org/1397293002 .

Remove crypto dependency and upgrade dart sdk dependency

6d8daee

move to pkg/test

70b655e

dartfmt

e09bb43

fix imports

d736278

Upgrade version in pubspec.yaml

cebf355

Make everything strong-mode clean (#8)

1dd290d

* Make everything strong-mode clean. Fix deprecation warnings for expectAsync. Make convert parameter for mapMap non-optional. * Run dartfmt

Prepare for new version, finish strong-mode cleaning and add 'analysi…

651717c

…s_options.yaml' file [email protected] Review-Url: https://codereview.chromium.org/2905613002 .

Fix USES_DYNAMIC_AS_BOTTOM violations (#10)

27b3033

Updates for Dart 2.0 core library changes (wave 2.2)

03775c8

add travis-ci support

e8c2715

Merge pull request #11 from dart-lang/corelib_2_2

d128106

Updates for Dart 2.0 core library changes (wave 2.2)

Fix strong mode issue with Dart2 (#13)

bdafaef

Fix screaming consts. (#14)

4a7df18

They're deprecated in the latest dev SDK, making Travis unhappy. Updated SDK constraint, .cast() was introduced in dev.22.0.

Fix range check. (#15)

5c9cf1f

Start and end are both inclusive in ByteRange, so it should be possible to create a range of 0-0. Fixes #6.

Dart 2 fix (#16)

5b7b7d7

Prepare release. (#17)

7f7b2da

Fix test in Dart 2 mode. (#19)

bf2e606

dartfmt

1b9734a

test: cleanup deprecated usages

1d7dbcd

analysis: disable implicit casts, enable other lints

3b52387

kevmoo and others added 19 commits January 23, 2019 07:19

Fix new analyzer errors

a522e0a

Enable and fix lints via dartfmt --fix

28b047e

fix some spelling error

45524b3

dartfmt fixes

53552b4

Remove forbidden headers

6ba5a97

Merge pull request #30 from jonasfj/fix-forbidden-headers

d9ef904

Remove forbidden headers

Send client library identifier

8f93e44

Merge pull request #32 from jonasfj/send-api-client-id

3cc6184

Send client library identifier

Prepare a new release

0a56fab

Merge pull request #33 from jonasfj/bump-version

98f4aca

Prepare a new release

Ignore uri not exists

f65b1dc

Merge pull request #34 from jonasfj/ignore-uri-not-exists

7dfe82c

Ignore uri not exists

Make ApiRequestError implement Exception and not Error.

6f5d2fc

The `dart:core` documentation clearly states errors represents a program failure that the programmer should have avoided. However, we throw a `DetailedApiRequestError` instance if the server responds 4xx/5xx.

Merge pull request #35 from jonasfj/implement-exception-not-error

cb970f9

Make `ApiRequestError` implement `Exception` and not `Error`.

Update pubspec.yaml for null safety.

7828545

Migrate to null safety.

d3f3bc9

Update CHANGEOG.md, pubspec.yaml, and .travis.yaml.

afc4e17

Merge pull request #36 from davidmorgan/null-safe

25a5a38

Migrate to null safety

merge in discoveryapis_common

3a0e56c

google-cla bot added the cla: no label Jan 22, 2021

kevmoo requested review from davidmorgan and jonasfj January 22, 2021 23:37

Fix lints and enable CI

c0dce01

kevmoo force-pushed the merge_discovery_common branch from 058d4d1 to c0dce01 Compare January 22, 2021 23:43

kevmoo closed this Jan 23, 2021

kevmoo deleted the merge_discovery_common branch January 25, 2021 14:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

merge discovery common #132

merge discovery common #132

kevmoo commented Jan 22, 2021

google-cla bot commented Jan 22, 2021

kevmoo commented Jan 22, 2021

google-cla bot commented Jan 22, 2021

kevmoo commented Jan 23, 2021

merge discovery common #132

merge discovery common #132

Conversation

kevmoo commented Jan 22, 2021

google-cla bot commented Jan 22, 2021

kevmoo commented Jan 22, 2021

google-cla bot commented Jan 22, 2021

kevmoo commented Jan 23, 2021