Skip to content

Pull1963refix#1964

Merged
Martii merged 2 commits intoOpenUserJS:masterfrom
Martii:pull1963refix
Jun 24, 2022
Merged

Pull1963refix#1964
Martii merged 2 commits intoOpenUserJS:masterfrom
Martii:pull1963refix

Conversation

@Martii
Copy link
Member

@Martii Martii commented Jun 24, 2022

See individual commits

Martii added 2 commits June 24, 2022 00:08
@Martii
Refix
ad3526c 
* Was testing out the min version and forgot to put the full path in... too late this evening heh.

Post OpenUserJS#1963

* NOTE: This starts to fail at *marked* btw with

> Uncaught ReferenceError: exports is not defined
>    at marked.cjs:14:23
@Martii
Use doc sample for now for *marked* instead of CJS *(the normal as re…
9c1774f 
…ferenced earlier)*

Post OpenUserJS#1963

NOTE:

* Preview is still damaged but closer... really consider changing deps if they can't get their act together or some missed change.
@Martii Martii added bug You've guessed it... this means a bug is reported. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. needs mitigation Needs additional followup. labels Jun 24, 2022
@Martii Martii merged commit e31abd0 into OpenUserJS:master Jun 24, 2022
@Martii Martii deleted the pull1963refix branch June 24, 2022 06:13
@Martii
Copy link
Member Author

Martii commented Jun 24, 2022 •
edited
Loading

Complete failure on production for previews... however partially working on development.... UGGH!

DEV:

dev

PRO:
pro

Retest with manual dep increment to 4.0.17 ... also FAIL...

@Martii
Copy link
Member Author

Martii commented Jun 24, 2022 •
edited
Loading

Forgot bootstrap-markdown is tied to this... need to see if parse is supported... could be failure on pro however doesn't explain why it's partially working in dev. Major UGHH!

Ref(s):

Martii added a commit that referenced this pull request Jun 24, 2022
@Martii
Update README.md
2f68096 
GH account name change detected.

Post #1964
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Jun 24, 2022
@Martii
Use fork for compatibility hack also dep update
de3dbcf 
Applies to OpenUserJS#1965 *(not closing in case production doesn't work since prior inconsistent test results... will close if applicable)*

Post OpenUserJS#1957 *(also may need to revert partial OpenUserJS#1964 shortly after full retest)*
@Martii Martii mentioned this pull request Jun 24, 2022
Merged
Martii added a commit that referenced this pull request Jun 24, 2022
@Martii
Use fork for compatibility hack also dep update (#1966)
1069e82 
Applies to #1965 *(not closing in case production doesn't work since prior inconsistent test results... will close if applicable)*

Post #1957 *(also may need to revert partial #1964 shortly after full retest)*

Auto-merge
@Martii Martii removed the needs mitigation Needs additional followup. label Jun 24, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

bug You've guessed it... this means a bug is reported. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment.

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Martii