Skip to content
This repository was archived by the owner on Nov 22, 2022. It is now read-only.

chore(package-name): update npm package name #23

Merged
elfrank merged 1 commit into from
Oct 1, 2019
Merged

chore(package-name): update npm package name #23

elfrank merged 1 commit into from
Oct 1, 2019

Conversation

@elfrank
Copy link
Collaborator

@elfrank elfrank commented Sep 30, 2019 •
edited
Loading

Brief Description

rename the npm package from three.js-ray-tracing-renderer to the new
name ray-tracing-renderer. We forgot to rename the project in a few
places after we renamed the repo.

Pull Request Guidelines

  • I have added pull requests labels which describe my contribution.
  • All existing tests passed.
    • I have added tests to cover my changes, of which pass.
  • I have compared the render output of my branch to master.
  • My code has passed the ESLint configuration for this project.
  • My change requires modifications to the documentation.
    • I have updated the documentation accordingly.

@elfrank
chore(package-name): update npm package name
4affb75 
rename the npm package from `three.js-ray-tracing-renderer` to the new
name `ray-tracing-renderer`. We forgot to rename the project in a few
places after we renamed the repo.
@elfrank elfrank self-assigned this Sep 30, 2019
@elfrank elfrank added the documentation Improvements or additions to documentation label Sep 30, 2019
jaxry
jaxry approved these changes Oct 1, 2019
View reviewed changes
Copy link
Contributor

@jaxry jaxry left a comment

Choose a reason for hiding this comment

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

LGTM

@elfrank elfrank merged commit 948bdae into master Oct 1, 2019
@elfrank elfrank deleted the chore-update-npm-package-name branch October 2, 2019 23:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@carlos-lopez-garces carlos-lopez-garces Awaiting requested review from carlos-lopez-garces

@lyonsno lyonsno Awaiting requested review from lyonsno

@santiagoroca santiagoroca Awaiting requested review from santiagoroca

1 more reviewer

@jaxry jaxry jaxry approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@elfrank elfrank

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@elfrank @jaxry