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

es-build #4

Merged
merged 4 commits into from
Jan 24, 2018
Merged

es-build #4

merged 4 commits into from
Jan 24, 2018

Conversation

kenwheeler
Copy link
Contributor

Addresses #2 cc @ryan-roemer

@ryan-roemer
Copy link
Contributor

I'm hitting:

npm run clean-es && babel src -d es --extensions ".ts,.tsx"

> urql@0.0.2 clean-lib /Users/rye/scm/fmd/urql
> rimraf lib

SyntaxError: src/components/client.tsx: Missing class properties transform.
  31 | 
  32 | export default class UrqlClient extends Component<ClientProps, ClientState> {
> 33 |   state = {
     |   ^
  34 |     fetching: false,
  35 |     loaded: false,
  36 |     error: null,

@ryan-roemer
Copy link
Contributor

UPDATE: It's apparently

    "@babel/plugin-proposal-class-properties": "^7.0.0-beta.38",

found this in babel/babel#2729 (comment)

@ryan-roemer
Copy link
Contributor

I've pushed a fix.

@ryan-roemer
Copy link
Contributor

You only need to specify what you're overriding, so I honed things down in 07d5505

"start": "webpack-dev-server --hot --inline",
"type-check": "tsc",
"lint": "eslint src",
"test": "jest",
"build": "babel src --out-dir lib --extensions \".ts,.tsx\""
"test": "jest"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Side note (not for now): Jest is picking up all directories, including es/ and failing on imports. We should restrict to src or lib or something

Copy link
Contributor

@ryan-roemer ryan-roemer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed a couple fixes so you should check those. Otherwise LGTM!!!

@kenwheeler kenwheeler merged commit eb07d21 into master Jan 24, 2018
@kitten kitten deleted the es-build branch May 26, 2018 04:33
kitten added a commit that referenced this pull request Feb 13, 2020
Replace GitHub Actions workflow with Travis CI
parkerziegler added a commit that referenced this pull request Feb 28, 2020
* Add support for accessing ctx in withUrqlClient.

* Update TS definitions. Add test of mergeExchanges.
parkerziegler added a commit that referenced this pull request Mar 1, 2020
* Initial commit.

* Check in initial work on README before beginning API rework.

* Add setup for Babel, Rollup, Jest. Begin working on NextJS example.

* Add next.config.js to example. Being to isolate peerDependencies.

* Check in progress on example and rollup config.

* Add bash script to assist in example dependency installation. Add API doscumentation and example README instructions.

* Add Pokemon type icons to example.

* README edits.

* Last cleanup before 0.1.0 publish.

* Check in initial CHANGELOG. Add npm badge to repo.

* Add example CodeSandbox link to repo.

* Improve publishing steps. Add formal LICENSE. Improve documentation. Add TypeScript definitions. (#2)

* Improve publishing steps. Add formal LICENSE. Improve documentation. Add TypeScript definitions.

* Remove declaration: true from tsconfig.

* PR feedback. Improve README language around automatic static optimization. Add CONTRIBUTING.md. Adjust ESLint settings.

* Changes for version 0.1.1

* v0.1.1

* Add support for accessing Next context to configure urql Client. (#4)

* Add support for accessing ctx in withUrqlClient.

* Update TS definitions. Add test of mergeExchanges.

* Add updated documentation on accessing ctx in withUrqlClient. Update CHANGELOG for v0.2.0 release.

* v0.2.0

* Prevent circular structure in getInitialProps. (#7)

* Prevent circular structure in getInitialProps.

* Update TS defs to better match wrapped components that implement their own getInitialProps.

* Final edits to index.d.ts.

* Prepare v0.2.1 release.

* v0.2.1

* Fix first argument type for withUrqlClient (#9)

* Prepare v0.2.2 release.

* v0.2.2

* Fix a few typos in the README.

* Fix support for using withUrqlClient with _app.js. (#13)

* Fix support for using withUrqlClient with _app.js

Co-authored-by: bmathews <brian.mathews@formidable.com>

* Update example READMEs.

* Final adjustments to index.d.ts

Co-authored-by: Brian Mathews <spoonart@gmail.com>

* Prepare v0.2.3 release.

* v0.2.3

* add urqlClient to Page Context (#15)

* add urqlClient to Page Context

* revert import ordering

* add urqClient to PageProps interface in index.d.ts

* move type to NextContextWithAppTree

* Prepare 0.2.4 release.

* Add urqlClient to mocked Next context object.

* v0.2.4

* chore: Convert example `1-with-urql-client` to TypeScript (#19)

* Add auto-generated tsconfig.json and next-env.d.ts

* Convert components and pages to TypeScript

* Use correct NextContext type

* Point TS compiler to next-urql at root of the repo

* Update example to use our custom NextContext

* chore: Convert example `2-with-_app.js` to TypeScript (#21)

* Convert example 2 to TypeScript

* Add `tsconfig.json` and `next-env.d.ts`
* JS -> TS conversion
* Add types for pokemon data
* Add type for Next App
* Refactor pokemon type mapping

* Use AppPropsType from Next

* Upgrade to Next 9 (#22)

* Upgrade to Next 9 (test fixes WIP)

* Remove deprecated `@types/next`
* Add `next@9.1.7` to devDependencies
* Replace `NextComponentClass` | `NextFC` union describing `Page` with
`NextPage`
* Rename `NextContextWithAppTree` to `NextUrqlContext`
* Make `NextUrqlContext` extend `NextPageContext` instead of
`NextContext`

* Import types directly from top-level next

* Coerce mock request object

* Fix withUrqlClient test config

* Coerce ctx into more accurate representation

* Prepare v0.2.5.

* v0.2.5

* Add instructions on using next-urql with ReasonML. (#28)

* Update for missing peer dependencies (#29)

* Add Travis CI. Autogenerate TS defs into dist. Add displayName for development environments. (#30)

* Add Travis CI. Autogenerate TS defs into dist. Add displayName for development environments.

* Fix issues with Rollup not transforming regeneratorRuntime in examples.

* Use correct name for formatting script in CI.

* Add in Buble.

* Preserve displayName in all environments. Add build badge to README.

* Add custom exchange example (#32)

* Add initial custom exchange example

* Remove console log

* Remove serialization of clientOptions in getInitialProps. (#33)

* Remove serialization of clientOptions in getInitialProps.

* Remove unnecessary console.log.

* Remove another console.log.

* Move mockMergeExchanges to upper describe block.

* Move beforeEach rendering component into relevant describe block.

* Remove test stage as Travis explicitly includes it.

* bump react-ssr-prepass to 1.1.2 to fix the suspense bug (#37)

* bump react-ssr-prepass to 1.1.1 to fix the suspense bug

* use 1.1.2 this includes some refactors

* Add better support for _app.js. Move types to separate module. (#38)

* Add better support for _app.js. Move types to separate module.

* Remove unnecessary type casting.

* Fix type issues in tests by relaxing types 😂

* Prepare v0.3.0 release.

* v0.2.0

* v0.3.0

* Demonstrate usage of getInitialProps in _app.js. (#40)

* Clean up old next-urql configs

* Fix build for next-urql

* Add correct next-urql test setup

* Lint next-urql files

* Update CHANGELOG and add changeset to next-urql

* Remove .gitignore in individual files. Edit README.

Co-authored-by: Kyle Alwyn <kjalwyn@gmail.com>
Co-authored-by: Brian Mathews <spoonart@gmail.com>
Co-authored-by: Björn Rave <36173920+BjoernRave@users.noreply.github.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Ayhan <ayhankuru@yandex.com.tr>
Co-authored-by: Jovi De Croock <decroockjovi@gmail.com>
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.

None yet

2 participants