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

chore: profiling #936

Closed
wants to merge 218 commits into from
Closed

chore: profiling #936

wants to merge 218 commits into from

Conversation

kellyjosephprice
Copy link
Collaborator

PR App Fix RM-XYZ

🧰 Changes

Describe your changes in detail.

🧬 QA & Testing

kellyjosephprice and others added 28 commits June 26, 2024 18:39
| [![PR App][icn]][demo] | RM-9917 |
| :--------------------: | :-----: |

## 🧰 Changes

Adds the position data to variable nodes, oops.

Useful in the editor.

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].

[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.62

### ✨ New & Improved

* mdx position ([#917](#917)) ([0e080fa](0e080fa))

<!--SKIP CI-->
| [![PR App][icn]][demo] | RM-9917 |
| :--------------------: | :-----: |

## 🧰 Changes

Adds `readme-variable` to the mdast for the editor to consume

I found myself transforming `mdx` nodes in the editor and decided I
should stick my existing convetion to do that in the library.

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].

[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.63

### ✨ New & Improved

* readme-variable ([#913](#913)) ([e3c4a5e](e3c4a5e))

<!--SKIP CI-->
[![PR App][icn]][demo] | Fix RM-10049
:-------------------:|:----------:

## 🧰 Changes

cleaner way to add/update image caption

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.64

### 🛠 Fixes & Updates

* add image component caption as children ([#918](#918)) ([59e1bd3](59e1bd3))

<!--SKIP CI-->
[![PR App][icn]][demo] | Fix RM-10134
:-------------------:|:----------:

## 🧰 Changes

Converts the rdmd-style `mdast` for the image and embed blocks to the
new MDX components

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.65

### ✨ New & Improved

* compatibility for rdmd image/embed components ([#919](#919)) ([e631c8f](e631c8f))

<!--SKIP CI-->
[![PR App][icn]][demo] | Fix RM-XYZ
:-------------------:|:----------:

## 🧰 Changes

migration diffs had a lot of discrepancies, tightening things up

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
Reverts #922'

forgot to push test fixes
[![PR App][icn]][demo] | Fix RM-XYZ
:-------------------:|:----------:

## 🧰 Changes

migration diffs had a lot of discrepancies, tightening things up (only v
minor changes)

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
[![PR App][icn]][demo] | Fix RM-XYZ
:-------------------:|:----------:

## 🧰 Changes

Adding `className` to the ImageBlock type for passing the `className`
from the component in the future. (also just adding trivial commit to
kick off release)

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
[![PR App][icn]][demo] | Fix RM-XYZ
:-------------------:|:----------:

## 🧰 Changes

unskipping and fixing one of the html block tests

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.66

### 🛠 Fixes & Updates

* html block test ([#927](#927)) ([7830a3d](7830a3d))

<!--SKIP CI-->
[![PR App][icn]][demo] | Fix RM-XYZ
:-------------------:|:----------:

## 🧰 Changes

who doesn't love a unit test

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.66

### ✨ New & Improved

* add another html block test ([#928](#928)) ([752be9e](752be9e))

### 🛠 Fixes & Updates

* html block test ([#927](#927)) ([7830a3d](7830a3d))

<!--SKIP CI-->
| [![PR App][icn]][demo] | RM-9815 |
| :--------------------: | :-----: |

## 🧰 Changes

Adds the heading ids to the tree when using `rmdx.hast`.

These ids are used in the apps indexing job.

## 🧬 QA & Testing

- [Broken on production][prod].
- [Working in this PR app][demo].

[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.67

### ✨ New & Improved

* hast heading ids ([#932](#932)) ([0075460](0075460))

### 🛠 Fixes & Updates

* **gha:** update github actions versions ([#930](#930)) ([e728a90](e728a90))

<!--SKIP CI-->
## Version 6.75.0-beta.68

### 🛠 Fixes & Updates

* **beta:** toc generation when using MDX components ([#933](#933)) ([c1408e3](c1408e3))

<!--SKIP CI-->
- [x] nix errant `console.log` per @jennspencer [on
#933](#933 (comment))
## Version 6.75.0-beta.69

### ✨ New & Improved

* force release ([d3e4da3](d3e4da3))

### 🛠 Fixes & Updates

* **beta:** nix errant log ([#934](#934)) ([04b55eb](04b55eb)), closes [#933](#933) [/github.com//pull/933#discussion_r1664579308](https://github.com/readmeio//github.com/readmeio/markdown/pull/933/issues/discussion_r1664579308)

<!--SKIP CI-->
[![PR App][icn]][demo] | Fix RM-10063
:-------------------:|:----------:

## 🧰 Changes

Unhandled nodes are throwing errors during migration. Offending node
types:

- [x] `div`
- [x] `embed` 
- [x] `figure`
- [x] `i`
- [x] `rdme-pin`
- [x] `yaml`

## 🧬 QA & Testing

You can look up the `Cannot handle unknown node <node>` errors from
Aaron's [migration
spreadsheet](https://docs.google.com/spreadsheets/d/1KVNWs8UeW_Gm7z2seR_ouTbHADXfZWLTUi87TBYB8ko/edit?gid=2109409773#gid=2109409773)
to find pages to test locally, if you want to use `make mdx` to swap out
the dist files in your `readme` repo.

You'll have to check out [his other
PR](https://github.com/readmeio/readme/pull/11999) with the migration
scripts first, which has directions for testing pages.

- [Broken on production][prod].
- [Working in this PR app][demo].


[demo]: https://markdown-pr-PR_NUMBER.herokuapp.com
[prod]: https://SUBDOMAIN.readme.io
[icn]:
https://user-images.githubusercontent.com/886627/160426047-1bee9488-305a-4145-bb2b-09d8b757d38a.svg
## Version 6.75.0-beta.70

### 🛠 Fixes & Updates

* compatibility for unknown nodes ([#929](#929)) ([5d21b54](5d21b54))

<!--SKIP CI-->
@kellyjosephprice kellyjosephprice deleted the chore/profiling branch October 29, 2024 17:32
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.

7 participants