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

Conversation

@zirkov
Copy link

@zirkov zirkov commented Jan 15, 2020

Summary

The continuation of #5 (can be closed when this PR is merged)

sous-chefs#566: caskroom/versions was moved. Tap homebrew/cask-versions instead

Description

Issues Resolved

Fixes <sous-chefs#566>:
`caskroom/versions` was moved. Tap `homebrew/cask-versions` instead

* Based on proposed changes from <sous-chefs#567>
* Switch homebrew tap to homebrew/cask-versions
* Make Homebrew Tap name an attribute to allow for other options

This file is used to list changes made in each version of the Java cookbook.

## Unreleased

Choose a reason for hiding this comment

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

Apprehensive about removing the Unreleased tag, because it's technically not released in the upstream and this was method by way of denoting that (and I think was a convention a year ago).
Might be worth keeping it or denoting Roblox Release or something

Copy link
Author

Choose a reason for hiding this comment

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

Oh, didn't realize that
Fixing

Copy link
Author

Choose a reason for hiding this comment

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

@Brantone Updated

Copy link

@Brantone Brantone left a comment

Choose a reason for hiding this comment

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

Minor tweak/thoughts on Changelog

default['java']['windows']['returns'] = 0
when 'mac_os_x'
default['java']['install_flavor'] = 'homebrew'
default['java']['homebrew']['tap'] = 'homebrew/cask-versions'

Choose a reason for hiding this comment

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

👍

@Brantone Brantone merged commit 052862b into Roblox:master Jan 15, 2020
@zirkov zirkov deleted the feature/homebrew-cask-versions-update branch January 15, 2020 22:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants