I've been able to read the values from a BME280 over I2C properly. However, I know that just because something functions doesn't mean it is done correctly. In the future I plan on have 2-3 devices over i2c (ADS1115, BME280, and an RTC).
I know that the ESP8266 has internal pull ups so perhaps the library activates them automatically?
As faar as I have experienced, and picked up here in Forum, I2C having only one master and one slave You don't need pullups. Using short cables 2 slaves outht to work. For more slaves pullup are used, some 10 kOhm if I remember.
@Paul__B
Maybe I referred to some very old advice picked up a long time ago, before the standardized modules of today, that might have pullup, existed. The discussion was using I2C or CAN buss, some 25 years ago.
Regarding long cables terminating resistors, pullups, do well regarding noice levels, capacitance etc. in my opinion.