Skip to content

Commit

Permalink
Move to pkg:lints
Browse files Browse the repository at this point in the history
  • Loading branch information
kevmoo committed Mar 11, 2022
1 parent 7d811f1 commit c7f0bb3
Show file tree
Hide file tree
Showing 33 changed files with 43 additions and 66 deletions.
8 changes: 5 additions & 3 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
## 9.0.2-dev

## 9.0.1
- Add `conclusion` property in class `CheckRun`

## 9.0.0

**Breaking change:** In the Gist class, the old type of files was
Expand All @@ -15,9 +20,6 @@ Map<String, GistFile>? files;

**Full Changelog**: https://github.com/SpinlockLabs/github.dart/compare/8.5.0...9.0.0

## 9.0.1
- Add `conclusion` property in class `CheckRun`

## 8.5.0

* Adds listing and creating PR Reviews, listing users in an org by @robrbecker in https://github.com/SpinlockLabs/github.dart/pull/287
Expand Down
2 changes: 1 addition & 1 deletion analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include: package:pedantic/analysis_options.yaml
include: package:lints/recommended.yaml

analyzer:
strong-mode:
Expand Down
15 changes: 8 additions & 7 deletions example/languages.dart
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import 'dart:html';

import 'package:github/github.dart';
import 'common.dart';

DivElement? tableDiv;
Expand Down Expand Up @@ -49,15 +48,17 @@ String generateMarkdown(int accuracy) {
final total = totalBytes(breakdown);
final data = breakdown.toList();

var md = '|Name|Bytes|Percentage|\n';
md += '|-----|-----|-----|\n';
var md = StringBuffer('''
|Name|Bytes|Percentage|
|-----|-----|-----|
''');
data.sort((a, b) => b[1].compareTo(a[1]));

data.forEach((info) {
for (final info in data) {
final String? name = info[0];
final int bytes = info[1];
final num percentage = (bytes / total) * 100;
md += '|$name|$bytes|${percentage.toStringAsFixed(accuracy)}|\n';
});
return md;
md.writeln('|$name|$bytes|${percentage.toStringAsFixed(accuracy)}|');
}
return md.toString();
}
1 change: 0 additions & 1 deletion example/organization.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import 'dart:async';
import 'dart:html';

import 'package:github/github.dart';
import 'common.dart';

DivElement? $output;
Expand Down
2 changes: 0 additions & 2 deletions example/pr.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
import 'dart:async';
import 'dart:html';

import 'package:github/github.dart';

import 'common.dart';

Future<void> main() async {
Expand Down
2 changes: 0 additions & 2 deletions example/readme.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
import 'dart:html';

import 'package:github/github.dart';

import 'common.dart';

Future<void> main() async {
Expand Down
2 changes: 0 additions & 2 deletions example/releases.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
import 'dart:html';

import 'package:github/github.dart';

import 'common.dart';

DivElement? releasesDiv;
Expand Down
6 changes: 2 additions & 4 deletions example/repos.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
import 'dart:async';
import 'dart:html';

import 'package:github/github.dart';

import 'common.dart';

DivElement? repositoriesDiv;
Expand All @@ -29,14 +27,14 @@ Future<void> main() async {
loadRepos();
});

sorts.keys.forEach((name) {
for (final name in sorts.keys) {
querySelector('#sort-$name')!.onClick.listen((event) {
if (_reposCache == null) {
loadRepos(sorts[name]);
}
updateRepos(_reposCache!, sorts[name]);
});
});
}
}

List<Repository>? _reposCache;
Expand Down
1 change: 0 additions & 1 deletion example/stars.dart
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import 'dart:html';

import 'package:github/github.dart';
import 'common.dart';

DivElement? $stars;
Expand Down
1 change: 0 additions & 1 deletion example/user_info.dart
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import 'dart:html';

import 'package:github/github.dart';
import 'common.dart';

DivElement? info;
Expand Down
2 changes: 0 additions & 2 deletions example/users.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
import 'dart:async';
import 'dart:html';

import 'package:github/github.dart';

import 'common.dart';

DivElement? usersDiv;
Expand Down
4 changes: 1 addition & 3 deletions lib/src/common/activity_service.dart
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:github/src/common/util/pagination.dart';
import 'package:github/src/common/util/utils.dart';
import 'package:http/http.dart' as http;

/// The [ActivityService] handles communication with activity related methods
Expand Down
2 changes: 1 addition & 1 deletion lib/src/common/authorizations_service.dart
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import 'dart:async';

import 'package:github/src/common.dart';
import 'package:github/src/common/util/pagination.dart';

/// The [AuthorizationsService] handles communication with authorizations related methods
/// of the GitHub API.
Expand Down
1 change: 0 additions & 1 deletion lib/src/common/checks_service.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import 'dart:convert';

import 'package:github/github.dart';
import 'package:github/src/common/util/utils.dart';

const _previewHeader = 'application/vnd.github.antiope-preview+json';

Expand Down
2 changes: 1 addition & 1 deletion lib/src/common/gists_service.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/util/pagination.dart';

/// The [GistsService] handles communication with gist
/// methods of the GitHub API.
Expand Down
2 changes: 1 addition & 1 deletion lib/src/common/git_service.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/util/pagination.dart';

/// The [GitService] handles communication with git related methods of the
/// GitHub API.
Expand Down
1 change: 0 additions & 1 deletion lib/src/common/github.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/util/utils.dart';
import 'package:http/http.dart' as http;
import 'package:http_parser/http_parser.dart' as http_parser;
import 'package:meta/meta.dart';
Expand Down
2 changes: 0 additions & 2 deletions lib/src/common/issues_service.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:github/src/common/util/pagination.dart';

/// The [IssuesService] handles communication with issues related methods of the
/// GitHub API.
Expand Down
1 change: 0 additions & 1 deletion lib/src/common/model/activity.dart
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:json_annotation/json_annotation.dart';

part 'activity.g.dart';
Expand Down
1 change: 0 additions & 1 deletion lib/src/common/model/git.dart
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:json_annotation/json_annotation.dart';

part 'git.g.dart';
Expand Down
1 change: 0 additions & 1 deletion lib/src/common/model/pulls.dart
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:json_annotation/json_annotation.dart';
import 'package:meta/meta.dart';

Expand Down
4 changes: 2 additions & 2 deletions lib/src/common/model/repos.dart
Original file line number Diff line number Diff line change
Expand Up @@ -309,8 +309,8 @@ class RepositorySlug {
String get fullName => '$owner/$name';

@override
bool operator ==(Object obj) =>
obj is RepositorySlug && obj.fullName == fullName;
bool operator ==(Object other) =>
other is RepositorySlug && other.fullName == fullName;

@override
int get hashCode => fullName.hashCode;
Expand Down
1 change: 0 additions & 1 deletion lib/src/common/model/repos_commits.dart
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:json_annotation/json_annotation.dart';

part 'repos_commits.g.dart';
Expand Down
3 changes: 1 addition & 2 deletions lib/src/common/orgs_service.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/util/pagination.dart';
import 'package:github/src/common/util/utils.dart';
import 'package:http/http.dart' as http;

/// The [OrganizationsService] handles communication with organization
Expand Down
11 changes: 5 additions & 6 deletions lib/src/common/pulls_service.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/util/pagination.dart';
import 'package:github/src/common/util/utils.dart';

/// The [PullRequestsService] handles communication with pull request
/// methods of the GitHub API.
Expand Down Expand Up @@ -182,9 +181,9 @@ class PullRequestsService extends Service {
Future<PullRequestReview> createReview(
RepositorySlug slug, CreatePullRequestReview review) {
return github.postJSON(
'/repos/${slug.fullName}/pulls/${review.pullNumber}/reviews',
body: GitHubJson.encode(review),
convert: (dynamic i) => PullRequestReview.fromJson(i))
as Future<PullRequestReview>;
'/repos/${slug.fullName}/pulls/${review.pullNumber}/reviews',
body: GitHubJson.encode(review),
convert: (dynamic i) => PullRequestReview.fromJson(i),
);
}
}
5 changes: 1 addition & 4 deletions lib/src/common/repos_service.dart
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/model/repos_releases.dart';
import 'package:github/src/common/model/users.dart';
import 'package:github/src/common/util/pagination.dart';
import 'package:github/src/common/util/utils.dart';
import 'package:http/http.dart' as http;

/// The [RepositoriesService] handles communication with repository related
Expand Down
3 changes: 1 addition & 2 deletions lib/src/common/search_service.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import 'dart:async';
import 'dart:convert';

import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:github/src/common/util/pagination.dart';

/// The [SearchService] handles communication with search related methods of
/// the GitHub API.
Expand Down
4 changes: 1 addition & 3 deletions lib/src/common/users_service.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
import 'dart:async';

import 'package:github/src/common.dart';
import 'package:github/src/common/model/users.dart';
import 'package:github/src/common/util/pagination.dart';
import 'package:github/src/common/util/utils.dart';
import 'package:http/http.dart' as http;

/// The [UsersService] handles communication with user related methods of the
Expand Down
2 changes: 2 additions & 0 deletions lib/src/common/util/utils.dart
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
// ignore_for_file: constant_identifier_names

import 'package:github/src/common.dart';
import 'package:meta/meta.dart';

Expand Down
1 change: 1 addition & 0 deletions lib/src/const/token_env_keys.dart
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// ignore: constant_identifier_names
const List<String> COMMON_GITHUB_TOKEN_ENV_KEYS = [
'GITHUB_ADMIN_TOKEN',
'GITHUB_DART_TOKEN',
Expand Down
4 changes: 2 additions & 2 deletions pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: github
version: 9.0.1
version: 9.0.2-dev
description: A high-level GitHub API Client Library that uses Github's v3 API
homepage: https://github.com/SpinlockLabs/github.dart

Expand All @@ -18,7 +18,7 @@ dev_dependencies:
build_test: any
build_web_compilers: any
json_serializable: ^6.0.0
lints: ^1.0.0
mockito: ^5.0.0
pedantic: ^1.10.0
test: ^1.16.0
yaml: ^3.0.0
10 changes: 5 additions & 5 deletions test/experiment/limit_pager.dart
Original file line number Diff line number Diff line change
Expand Up @@ -5,21 +5,21 @@ void main() {
print(solve(201));
}

const int MAX_PER_PAGE = 100;
const int ACCURACY_RANGE = 5;
const int maxPerPage = 100;
const int accuracyRange = 5;

/// Solves the most efficient way to fetch the number of objects [limit] with the least requests.
PaginationInformation solve(int limit) {
if (limit < 0) {
throw RangeError('limit cannot be less than zero (was $limit)');
}

if (limit < MAX_PER_PAGE) {
if (limit < maxPerPage) {
return PaginationInformation(limit, 1, limit);
}

if ((limit % MAX_PER_PAGE) == 0) {
return PaginationInformation(limit, limit ~/ MAX_PER_PAGE, MAX_PER_PAGE);
if ((limit % maxPerPage) == 0) {
return PaginationInformation(limit, limit ~/ maxPerPage, maxPerPage);
}

const itemsPerPage = 100;
Expand Down
2 changes: 2 additions & 0 deletions test/src/mocks.mocks.dart
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,12 @@ import 'package:mockito/mockito.dart' as _i1;

// ignore_for_file: avoid_redundant_argument_values
// ignore_for_file: avoid_setters_without_getters
// ignore_for_file: camel_case_types
// ignore_for_file: comment_references
// ignore_for_file: implementation_imports
// ignore_for_file: invalid_use_of_visible_for_testing_member
// ignore_for_file: prefer_const_constructors
// ignore_for_file: unnecessary_overrides
// ignore_for_file: unnecessary_parenthesis

class _FakeClient_0 extends _i1.Fake implements _i2.Client {}
Expand Down

0 comments on commit c7f0bb3

Please sign in to comment.