Skip to content

Conversation

alex-james-dev
Copy link
Contributor

This is the first pull request for the java_http package! 🎉 Relevant issue: #957.

In this PR I created a new Dart package called java_http using the Dart create command with the package template. This package was placed in the pkgs directory along with the other package:http clients. Functionality wise it doesn't do anything yet. It is the basic structure of a Dart package to build upon with future pull requests.

The main changes made to the code generated by the Dart create command were:

  • Set the version to 0.0.1 in CHANGELOG.md and pubspec.yaml.
  • Set publish_to: none in pubspec.yaml.
  • Set SDK to ^3.0.0 in pubspec.yaml. This is the same as cronet_http, cupertino_http and http.

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Create a new Dart package in pkgs called java_http using the Dart create
command and the package template.
Remove the .gitignore file in java_http that was generated automatically
by the Dart create command.
Currently, we don't need a java_http specific .gitignore file because
the .gitignore file in the parent directory contains all the required
gitignore patterns.
Remove the analysis_options.yaml file in java_http that was generated
automatically by the Dart create command. Then the Dart analyzer will
use the analysis_options.yaml file in the parent directory.
Update the java_http README.md which was automatically generated by the
Dart create command.
Add a comment to the java_http source files that were generated
automatically by the Dart create command with the package template.
@brianquinlan brianquinlan self-requested a review June 23, 2023 16:04
@brianquinlan brianquinlan self-assigned this Jun 23, 2023
@brianquinlan brianquinlan merged commit 067bff3 into dart-lang:master Jun 26, 2023
@alex-james-dev alex-james-dev deleted the create-java-http-package branch June 28, 2023 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants